Minor test edits #18
Merged
Minor test edits #18
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* More robust finding of blas in cmake * Update * Update * Fix typo * Update * Update * Update * Update * Update
Comment had the answer for the PSE test. Removed the periodic correction part
… dissipation test.
rykerfish
added a commit
that referenced
this pull request
Jul 18, 2024
commit ac2b405 Author: Ryker Fish <rfish@mines.edu> Date: Tue Jul 16 19:41:16 2024 -0600 regenerated data w new defaults after verifying against old repository commit 07b4794 Author: Ryker Fish <rfish@mines.edu> Date: Tue Jul 16 19:05:08 2024 -0600 added parameter to initialize that sets whether the box size can get altered or not commit 447e543 Author: Ryker Fish <rfish@mines.edu> Date: Tue Jul 16 18:22:27 2024 -0600 added mode that preserves L commit 74ec436 Author: Ryker Fish <rfish@mines.edu> Date: Tue Jul 16 17:23:24 2024 -0600 checkpoint commit- adding functionality to have parameter select that modifies the box size and another version that preserves it commit 1113390 Author: Ryker Fish <rfish@mines.edu> Date: Wed Jul 10 14:59:33 2024 -0600 the changes to DPStokes parameters seem to made the mobility matrix less symmetric commit 800f634 Author: Ryker Fish <rfish@mines.edu> Date: Wed Jul 10 14:56:08 2024 -0600 had to change the ref file to use the right positions. nbody test working. commit 830866e Author: Ryker Fish <rfish@mines.edu> Date: Mon Jul 8 18:46:51 2024 -0600 the two pair DPStokes tests are passing. currently having issues with the nbody test commit 9b3a4d3 Author: Ryker Fish <rfish@mines.edu> Date: Fri Jun 28 14:19:28 2024 -0600 self mobility tests done commit c4f2664 Author: Ryker Fish <rfish@mines.edu> Date: Wed Jun 12 15:31:55 2024 -0600 larger safety factor near open boundary for dpstokes commit 5da7450 Author: Ryker Fish <rfish@mines.edu> Date: Wed Jun 12 12:15:33 2024 -0600 modify how N gets rounded in all dimensions to be fft friendly commit 51db36a Author: Ryker Fish <rfish@mines.edu> Date: Wed Jun 12 11:58:39 2024 -0600 changed selection of grid parameters to not modify h. instead, modify the box size to make L/h an integer commit 5127394 Author: Ryker Fish <rfish@mines.edu> Date: Thu Jun 6 14:07:01 2024 -0600 changed test pass threshold commit 4fb82bf Author: Ryker Fish <rfish@mines.edu> Date: Thu Jun 6 13:42:52 2024 -0600 regenerated all ref data using libmobility implementations in single precision commit 4392903 Author: Ryker Fish <rfish@mines.edu> Date: Thu Jun 6 13:25:04 2024 -0600 regenerated refs using gpu implementations from dpstokes library. tests are now passing if the same kernel parameters are used as the ref implementation commit 1ce606a Author: Ryker Fish <rfish@mines.edu> Date: Thu May 30 16:06:32 2024 -0600 fixed bug in nbody wall correction that came from normalizing by a factor of 1/8pi in the original implementation instead of 1/6pi in the Swan paper. commit edf78f5 Author: Ryker Fish <rfish@mines.edu> Date: Thu May 30 16:05:21 2024 -0600 switched all tests to w=4 kernels, added nbody to pair mobility tests commit c575ef2 Author: Ryker Fish <rfish@mines.edu> Date: Wed May 29 15:11:48 2024 -0600 change dpstokes to use w=4 kernel by default for forces commit 070728b Author: Ryker Fish <rfish@mines.edu> Date: Tue May 28 16:57:09 2024 -0600 started pair mobility test commit ca4e1c7 Author: Ryker Fish <rfish@mines.edu> Date: Sat May 18 12:19:26 2024 -0600 combined self mobility tests into one function commit e90e9e2 Author: Ryker Fish <rfish@mines.edu> Date: Fri May 17 20:45:24 2024 -0600 bug fix in NBody with wall correction- test now passing. the fix uses the correct constants from the Swan correction paper- the previous implementation was using constants based on a self mobility constant of 1/(8*pi*eta*a) instead of 1/(6*pi*eta*a) commit 98e2ae0 Author: Ryker Fish <rfish@mines.edu> Date: Fri May 17 15:37:25 2024 -0600 changed dpstokes test to compare to dpstokes reference and added a not-working nbody test commit 7aaa0b1 Merge: 79fe6a7 d6144a5 Author: Ryker <rfish@mines.edu> Date: Mon May 13 14:25:35 2024 -0600 Merge pull request #18 from stochasticHydroTools/minor_test_edits Minor test edits commit d6144a5 Author: Ryker Fish <rfish@mines.edu> Date: Thu May 9 15:40:00 2024 -0600 removed test cases for small numbers of particles for the fluctuation dissipation test. commit 0e368d4 Author: Ryker <rfish@mines.edu> Date: Thu May 9 14:37:44 2024 -0600 Edit comment on nbody test Comment had the answer for the PSE test. Removed the periodic correction part commit 79fe6a7 Author: Ryker Fish <rfish@mines.edu> Date: Thu May 9 12:47:42 2024 -0600 now testing if mobility is 0 on the bottom wall commit 3c5002e Author: Ryker Fish <rfish@mines.edu> Date: Wed May 8 19:02:35 2024 -0600 possible correction to DPStokes code to add a buffer the top instead of the bottom when there is a wall commit c66303c Author: Ryker Fish <rfish@mines.edu> Date: Wed May 8 18:27:40 2024 -0600 selfmobility test working in comparison to refernce results for one and two wall cases commit 70aa329 Author: Ryker Fish <rfish@mines.edu> Date: Wed May 8 17:12:19 2024 -0600 mobility doesn't go to zero at the wall unless this section is commented out. commit fe2c964 Author: Ryker Fish <rfish@mines.edu> Date: Wed May 8 16:37:04 2024 -0600 first pass at computing translational self mobility components for bottom wall scenario
RaulPPelaez
added a commit
that referenced
this pull request
Sep 19, 2024
* Add torques to pythonify * Add torque to mobility interface * Add torques to DPStokes * Add torques to NBody * Add torques to PSE * Add torques to selfMobility * Update pythonify * Update python example * Update cpp example * Update test * Update test * Update test * Update test * Update test * Typo * Fix initialize * Typo * Update pythonify * Update pythonify * Typo * Update sqrtM * Update * Update * Typo * Change name * Update * Always send valid pointers to hydrodynamicVelocities * dont complain if receiving angular * Typo in tests * added torques to selfmobility test * angular mobility matrix test setup * added torques to selfmobility * updated interface so that the linear fluctuation dissipation test is passing. the problem was that a zero vector of angular velocities was always being passed into sqrtMdotW inside pythonify.h so solvers where torques weren't implemented would always throw the error, even though we were only trying to compute with forces. * removed unnecessary allocation for when torques aren't being used in sqrtMdotW * Squashed commit of the following: commit ac2b405 Author: Ryker Fish <rfish@mines.edu> Date: Tue Jul 16 19:41:16 2024 -0600 regenerated data w new defaults after verifying against old repository commit 07b4794 Author: Ryker Fish <rfish@mines.edu> Date: Tue Jul 16 19:05:08 2024 -0600 added parameter to initialize that sets whether the box size can get altered or not commit 447e543 Author: Ryker Fish <rfish@mines.edu> Date: Tue Jul 16 18:22:27 2024 -0600 added mode that preserves L commit 74ec436 Author: Ryker Fish <rfish@mines.edu> Date: Tue Jul 16 17:23:24 2024 -0600 checkpoint commit- adding functionality to have parameter select that modifies the box size and another version that preserves it commit 1113390 Author: Ryker Fish <rfish@mines.edu> Date: Wed Jul 10 14:59:33 2024 -0600 the changes to DPStokes parameters seem to made the mobility matrix less symmetric commit 800f634 Author: Ryker Fish <rfish@mines.edu> Date: Wed Jul 10 14:56:08 2024 -0600 had to change the ref file to use the right positions. nbody test working. commit 830866e Author: Ryker Fish <rfish@mines.edu> Date: Mon Jul 8 18:46:51 2024 -0600 the two pair DPStokes tests are passing. currently having issues with the nbody test commit 9b3a4d3 Author: Ryker Fish <rfish@mines.edu> Date: Fri Jun 28 14:19:28 2024 -0600 self mobility tests done commit c4f2664 Author: Ryker Fish <rfish@mines.edu> Date: Wed Jun 12 15:31:55 2024 -0600 larger safety factor near open boundary for dpstokes commit 5da7450 Author: Ryker Fish <rfish@mines.edu> Date: Wed Jun 12 12:15:33 2024 -0600 modify how N gets rounded in all dimensions to be fft friendly commit 51db36a Author: Ryker Fish <rfish@mines.edu> Date: Wed Jun 12 11:58:39 2024 -0600 changed selection of grid parameters to not modify h. instead, modify the box size to make L/h an integer commit 5127394 Author: Ryker Fish <rfish@mines.edu> Date: Thu Jun 6 14:07:01 2024 -0600 changed test pass threshold commit 4fb82bf Author: Ryker Fish <rfish@mines.edu> Date: Thu Jun 6 13:42:52 2024 -0600 regenerated all ref data using libmobility implementations in single precision commit 4392903 Author: Ryker Fish <rfish@mines.edu> Date: Thu Jun 6 13:25:04 2024 -0600 regenerated refs using gpu implementations from dpstokes library. tests are now passing if the same kernel parameters are used as the ref implementation commit 1ce606a Author: Ryker Fish <rfish@mines.edu> Date: Thu May 30 16:06:32 2024 -0600 fixed bug in nbody wall correction that came from normalizing by a factor of 1/8pi in the original implementation instead of 1/6pi in the Swan paper. commit edf78f5 Author: Ryker Fish <rfish@mines.edu> Date: Thu May 30 16:05:21 2024 -0600 switched all tests to w=4 kernels, added nbody to pair mobility tests commit c575ef2 Author: Ryker Fish <rfish@mines.edu> Date: Wed May 29 15:11:48 2024 -0600 change dpstokes to use w=4 kernel by default for forces commit 070728b Author: Ryker Fish <rfish@mines.edu> Date: Tue May 28 16:57:09 2024 -0600 started pair mobility test commit ca4e1c7 Author: Ryker Fish <rfish@mines.edu> Date: Sat May 18 12:19:26 2024 -0600 combined self mobility tests into one function commit e90e9e2 Author: Ryker Fish <rfish@mines.edu> Date: Fri May 17 20:45:24 2024 -0600 bug fix in NBody with wall correction- test now passing. the fix uses the correct constants from the Swan correction paper- the previous implementation was using constants based on a self mobility constant of 1/(8*pi*eta*a) instead of 1/(6*pi*eta*a) commit 98e2ae0 Author: Ryker Fish <rfish@mines.edu> Date: Fri May 17 15:37:25 2024 -0600 changed dpstokes test to compare to dpstokes reference and added a not-working nbody test commit 7aaa0b1 Merge: 79fe6a7 d6144a5 Author: Ryker <rfish@mines.edu> Date: Mon May 13 14:25:35 2024 -0600 Merge pull request #18 from stochasticHydroTools/minor_test_edits Minor test edits commit d6144a5 Author: Ryker Fish <rfish@mines.edu> Date: Thu May 9 15:40:00 2024 -0600 removed test cases for small numbers of particles for the fluctuation dissipation test. commit 0e368d4 Author: Ryker <rfish@mines.edu> Date: Thu May 9 14:37:44 2024 -0600 Edit comment on nbody test Comment had the answer for the PSE test. Removed the periodic correction part commit 79fe6a7 Author: Ryker Fish <rfish@mines.edu> Date: Thu May 9 12:47:42 2024 -0600 now testing if mobility is 0 on the bottom wall commit 3c5002e Author: Ryker Fish <rfish@mines.edu> Date: Wed May 8 19:02:35 2024 -0600 possible correction to DPStokes code to add a buffer the top instead of the bottom when there is a wall commit c66303c Author: Ryker Fish <rfish@mines.edu> Date: Wed May 8 18:27:40 2024 -0600 selfmobility test working in comparison to refernce results for one and two wall cases commit 70aa329 Author: Ryker Fish <rfish@mines.edu> Date: Wed May 8 17:12:19 2024 -0600 mobility doesn't go to zero at the wall unless this section is commented out. commit fe2c964 Author: Ryker Fish <rfish@mines.edu> Date: Wed May 8 16:37:04 2024 -0600 first pass at computing translational self mobility components for bottom wall scenario * modified wall tests to still pass in single precision by using variable tolerance based on machine epsilon * modified how symmetric matrix has to be for fluctuation dissipation tests to accomodate dpstokes parameters * changed to only allocate angular vec if torques are provided so that PSE and NBody work when torques aren't provided * interface test for mdot with torques * updated all interfaces to accept torques * started implementing NBody torques. naive algorithm works for main diagonal blocks- still need to do off diagonal blocks * fast algorithm now working for main diagonal * updated angular test to use different torques and forces * added off-diagonal blocks for torques and fixed bug in memory allocation for fast algorithm when using torques * added torque test for when forces are zero with nonzero torques * added test for freespace nbody with torques * bug fix in nbody UT component * add test for PSE not supporting torques * added torque parameters * add DPStokes to mob. matrix tests and remove PSE from tests involving torques * self mobility tests for DPstokes with torques * dpstokes pair wall tests complete * rewrite linear wall test to be more like angular test * fix bug in NBody bloc kalgorithm * added parameterization to test different nbody algorithms to show bug * bug fix- thanks Raul! * add block algorithm back to nbody tests * added torques to nbody block * added nbody wall kernels for self mobilities. still need to do pair mobilities * checkpoint commit for today, stuff still kinda broken but I think I know why * simplified tests since ref files have changed * added wall test for particle offset in y direction * wall kernels for torques in Nbody now working after bug fixes * added some comments on the subtle parts of the NBody wall kernels * minor optimization changes (went division hunting) and standardized notation between functions * add line to example to indicate to compile in double if necessary * change python example function calls to reflect interface * Set pybind11 as required * Take some code out of the PYTHONIFY macro * Format * Fix module name * Fix typo in docstring * Set C++ std to 17 * Update docstrings * Check if torques are being passed when needsTorques=False * Fix test not setting needsTorque=True * Install test scripts to /share/ * Add a different test per file * Improve mkl detection --------- Co-authored-by: Ryker Fish <rfish@mines.edu>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.