-
Notifications
You must be signed in to change notification settings - Fork 139
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
adding qmcfinitesize tool #2329
Merged
Merged
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
2b977cf
broken, but initial update of qmcfinite to new code
camelto2 545c267
edits to make qmcfinitesize up to date with develop
camelto2 638152d
changes to output formatting
camelto2 519ba94
Merge branch 'develop' into qmcFScorr
camelto2 2a3387b
clang-formatting
camelto2 0150c9b
removed empty file SkParserHDF5.h
camelto2 0326b05
remove unused code
camelto2 9493edb
improved spherical grid and handling cases outside spline grid
camelto2 c741895
add warning for S(k) not being convegred
camelto2 901c0ef
small fix
camelto2 4e69879
Merge remote-tracking branch 'upstream/develop' into qmcFScorr
camelto2 d21f65b
isolate qmcfinitesize in cmake
camelto2 3f34114
small change to spherical_grid
camelto2 4575b2b
refactoring for error estimates
camelto2 273c40b
add stats function, and get potential corr ready for bootstrap
camelto2 971336b
FS corrections with bootstrapped error estimate
camelto2 2b180af
Merge remote-tracking branch 'upstream/develop' into qmcFScorr
camelto2 920fe12
clang format and remove unused CellParserBase
camelto2 e425e64
fix to rng to work with mixed precision build
camelto2 65e8602
removed empty CMakeLists
camelto2 f5a36af
Merge remote-tracking branch 'upstream/develop' into qmcFScorr
camelto2 f852a4c
initial unit test architecture for QMCTools and QMCFiniteSize
camelto2 c0b999f
adding unit test, interface to vlr_k in LRHandlers
camelto2 d06cda6
Merge remote-tracking branch 'upstream/develop' into qmcFScorr
camelto2 327a9e2
clang-formatting and copyright updates on touched files
camelto2 34a226f
Merge branch 'develop' into qmcFScorr
prckent dcada08
fix constructor initialization order and no return type
camelto2 15cf71f
Merge remote-tracking branch 'upstream/develop' into qmcFScorr
camelto2 a506037
Merge remote-tracking branch 'origin/qmcFScorr' into qmcFScorr
camelto2 5d9e634
to trigger the CI
PDoakORNL 8f7dba4
fix compiler warnings for return types and QMCFiniteSize initializatino
camelto2 2f8ad91
Merge branch 'qmcFScorr' of https://github.com/camelto2/qmcpack into …
camelto2 2113d4c
to trigger CI
PDoakORNL 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
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
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
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.
Could you check if the dependencies here are necessary?
Do you need qmcwfs or just einspline?
Is qmc target really necessary?
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.
at the moment, it only needs einspline. this will change in the future when a kinetic energy FS correction is added, which will depend on the 2body jastrow and a kspace jastrow. if it is preferable to just add the various .cpp files in FSSRCS that each thing depends on I can do that instead as opposed to targetting libraries that include those dependencies
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.
Let us put einspline and change it to qmcwfs when actual WFS dependence comes in.
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.
For user friendliness, this tool should work with any QMCPACK xml input file. Unfortunately, even if you're not using any wavefunction specific information, the system initialization path that allows you to grab a particle set and box from ESHDF file, in the event it's not specified in the XML, needs to go through the wavefunction builder. So I think that qmcwfs is a necessary dependency right now.
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.
Thanks. I see. The dependency is needed right now.