-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add plugin for extended sampling #231
Draft
HomesGH
wants to merge
83
commits into
master
Choose a base branch
from
chemPot_sampling
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 45 commits
Commits
Show all changes
83 commits
Select commit
Hold shift + click to select a range
e471b72
First version of direct sampling of chemical potential
HomesGH b69d4a0
Fixes in ChemPotSampling
HomesGH 221cf37
Fixes in chem. pot. sampling
HomesGH 9df491a
Fixes in DriftCtrl; Re-enable drift control in all directions
HomesGH b32c726
Get rid of comparison warning
HomesGH 114d0be
Add temperature sampling including drift
HomesGH 1d07e55
Add documentation and fixes
HomesGH a6ffa33
Minor changes for improved usability
HomesGH 0f54e50
Fix in ChemPotSampling
HomesGH bb9c0f3
Add documentation
HomesGH 7324f17
Fix to avoid zero-division
HomesGH 0da5192
Fix to have at least one test particle per direction and process
HomesGH 87c20ac
Add documentation of getEnergy() method
HomesGH 3fb1fc5
Fix in MMPLD writer to get correct max. number of time steps
HomesGH c02c78a
Add possibility to use random insertion and sample every nth step; Fi…
HomesGH f6853bb
Get rid of warning
HomesGH 355ba31
Clean up and restructuring; include OPENMP
HomesGH cb12427
Fix in chemical potential sampling
HomesGH 84debc3
Minor improvements in DriftCtrl
HomesGH cdd1b25
Add python script to calculate shear viscosity for LJTS using Lautens…
HomesGH 8d8e1e5
Get rid of LGTM python warning
HomesGH ba6aa14
Changes in the implementation of the PeTS to get chemical potential (…
HomesGH e022806
Merge branch 'chemPot_sampling' of https://github.com/ls1mardyn/ls1-m…
HomesGH 350fb4a
Add function to convert gibbs free energy values
HomesGH f9ea383
Add checks to prevent invalid operations
HomesGH 0446f65
Abort simulation if xml-includes were not found
HomesGH e30785e
Get rid of LGTM warning
HomesGH 214db73
Add debug output
HomesGH 364bd01
Merge branch 'chemPot_sampling' of https://github.com/ls1mardyn/ls1-m…
HomesGH 0554bfb
Extended plugin to sample all relevant data
HomesGH 2d229f0
Minor fixes in sampling plugin
HomesGH c53f4dc
Fix in DriftCtrl to use velocities instead of momenta
HomesGH 20ef5cc
Fix in DriftCtrl to work with all components (cid=0)
HomesGH ed94b8b
Make stop work for RegionSampling; Clearify output in MettDaemon
HomesGH 922f1df
Include git information in executable as it was done with cmake
HomesGH 0c102d8
Extend and improve sampling of profile data
HomesGH c51d7ca
Enable direct sampling of energy flux when using the legacy cell proc…
HomesGH de0101b
Add calculation of all virial elements using vectorization
HomesGH 2301ed8
Add sampling of higher moments
HomesGH 7116946
Add possibility to specify direction of DriftCtrl
HomesGH 8bbb9b2
Update version file if commit hash changed
HomesGH 9504d41
Fix in full virial calculation with vectorization
HomesGH bf0efa0
Remove debug output
HomesGH 34cb6ce
Fixes for extended virial calculation
HomesGH 04dc4a5
Minor fixes in ExtendedSampling
HomesGH 4beadf0
Extension and fix of sampling of higher moments
HomesGH 6e71b7a
Realize suggested changes
HomesGH aba3e78
Minor fixes in ExtendedSampling plugin
HomesGH 667ab7d
Minor fixes in ExtendedSampling plugin
HomesGH c83fc0b
Implement suggested changes
HomesGH d6ae1bc
Fix error in shared pointer in ExtendedSampling plugin
HomesGH db559d1
Add range specification to gamma writer
HomesGH f208b49
Fixes in GammaWriter and sampling of all components as one
HomesGH 51c5dfc
Fix in MettDeamon and EPS regarding output
HomesGH 718687f
Add method to get upot correction per molecule; Fixes
HomesGH 8f6ed79
Add documentation of ExtendedProfileSampling plugin
HomesGH d115261
Add unit test for ExtendedSampling plugin and fix some warnings
HomesGH 1b6d9e4
Consider sample in EPS only if enough particles
HomesGH 1451948
Add sampling of massflux and fix in force sampling
HomesGH 28e571d
Fixes in ESP, TemperatureControl and DriftControl
HomesGH e9f4a41
Fix sampling of chem. pot. for LRC
HomesGH 1dee5ae
Fix in EPS plugin for multi-atom molecules
HomesGH 298ce87
Merge remote-tracking branch 'origin/master' into chemPot_sampling
HomesGH 9468c86
Fix in GammaWriter
HomesGH 2bf97db
Merge remote-tracking branch 'origin/master' into chemPot_sampling
HomesGH 452abb8
Fix in HigherMoments of EPS
HomesGH 82862b2
Merge remote-tracking branch 'origin/master' into chemPot_sampling
HomesGH 141bfec
Fix in sampling of higher moments
HomesGH c0e4044
Add orientation angle to sampling
HomesGH 9a51dd2
Merge branch 'chemPot_sampling' of https://github.com/ls1mardyn/ls1-m…
HomesGH e1c300f
Fix default of Quaternions (which was not normalized)
HomesGH de34309
Change orientation from angle to order parameter
HomesGH d738d77
Fix regarding quaternions
HomesGH 745a9e7
Normalize quaternions in test input files
HomesGH 20976e2
Normalize quaternions during read
HomesGH 8f66703
Add script to run validation tests
HomesGH 1444f41
Merge branch 'master' into chemPot_sampling
HomesGH 90e835c
Add std::
HomesGH 9d9dd0e
Performance improvements
HomesGH 07acfdc
Fix in ExtProfileSampling
HomesGH c3affd0
Minor fixes
HomesGH 334a32d
Fix in EPS
HomesGH d2b9583
Fix in higher moments and unit test
HomesGH File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -172,9 +172,18 @@ INCLUDES = -I$(SRCDIR) -isystem $(SRCDIR)/../libs/rapidxml -isystem $(SRCDIR)/.. | |
BINARY_BASENAME = MarDyn | ||
ifneq ("$(wildcard $(SRCDIR)/../.git)","") | ||
$(info Building from a git version.) | ||
GIT_VERSION = $(shell git rev-parse --short HEAD | sed -e "s/:/-/") | ||
ifneq ($(GIT_VERSION),) | ||
BINARY = $(BINARY_BASENAME)_$(GIT_VERSION).$(PARTYPE)_$(TARGET)_$(VECTORIZE_CODE) | ||
GIT_VERSION_BRANCH = $(shell git rev-parse --abbrev-ref HEAD | sed -e "s/:/-/") | ||
GIT_VERSION_HASH = $(shell git rev-parse --short HEAD | sed -e "s/:/-/") | ||
GIT_VERSION_IS_DIRTY = $(shell git diff --quiet || echo "_dirty") | ||
GIT_VERSION_HASH_CURRENT = $(shell grep "MARDYN_VERSION_HASH =" MarDyn_version.h 2> /dev/null | awk -F '"' '{print $$2}' | tr -d '_') | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I guess this simulates the |
||
ifneq ($(GIT_VERSION_HASH),$(GIT_VERSION_HASH_CURRENT)) | ||
$(shell rm -f MarDyn.o) | ||
$(shell sed "s=@MarDyn_VERSION_BRANCH@=$(GIT_VERSION_BRANCH)=g" MarDyn_version.h.in > MarDyn_version.h) | ||
$(shell sed -i "s=@MarDyn_VERSION_HASH@=$(GIT_VERSION_HASH)=g" MarDyn_version.h) | ||
$(shell sed -i "s=@MarDyn_VERSION_IS_DIRTY@=$(GIT_VERSION_IS_DIRTY)=g" MarDyn_version.h) | ||
endif | ||
ifneq ($(GIT_VERSION_HASH),) | ||
BINARY = $(BINARY_BASENAME)_$(GIT_VERSION_HASH).$(PARTYPE)_$(TARGET)_$(VECTORIZE_CODE) | ||
else | ||
BINARY = $(BINARY_BASENAME).$(PARTYPE)_$(TARGET)_$(VECTORIZE_CODE) | ||
endif | ||
|
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
187 changes: 138 additions & 49 deletions
187
src/bhfmm/cellProcessors/VectorizedChargeP2PCellProcessor.cpp
Large diffs are not rendered by default.
Oops, something went wrong.
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why the
sed
command? AFAIK neither tags, branch names, nor hashes can contain:
. This would also be inconsistent to thecmake
behavior.