-
Notifications
You must be signed in to change notification settings - Fork 620
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
Improved sampling algorithm for mcsolve #2218
Merged
Merged
Changes from 8 commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
499d44d
improved sampling algorithm for mcsolve working in initial tests
dkweiss31 9101b74
avoid code duplication by utilizing existing classes, taking care not…
dkweiss31 f034705
black
dkweiss31 124eabc
small bug fixes in average_states, added tests leveraging existing mc…
dkweiss31 5284162
docstrings, updated user guide
dkweiss31 1fff804
small additions to docs
dkweiss31 f29d6b8
added towncrier changelog
dkweiss31 966f495
Merge branch 'qutip:master' into mcsolve_efficienvy_v2
dkweiss31 8ea0c56
better parametrization of tests
dkweiss31 be9a6e9
Revert "black"
dkweiss31 3a43fd0
eliminated unnecesary function in MCSolverImprovedSampling, more cons…
dkweiss31 9dc74d0
added public classes to __all__, ensured compliance with pycodestyle
dkweiss31 41e845e
Moved improved_sampling into options, merged MCSolverImprovedSampling…
dkweiss31 7e181d5
updated documentation with an example plot showing usage of improved_…
dkweiss31 7198a45
small docs bug fix
dkweiss31 b38134e
Update doc/guide/dynamics/dynamics-monte.rst
dkweiss31 96d7ad6
Update qutip/solver/result.py
dkweiss31 f18e078
using decorator @property to set resultclass based on self.options["i…
dkweiss31 851d7e6
Merge remote-tracking branch 'origin/mcsolve_efficienvy_v2' into mcso…
dkweiss31 2984036
small clarity changes to docs
dkweiss31 File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
Improved sampling algorithm for mcsolve |
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.
I feel this should be in options.
options
include modification to the algorithm that do not change the physic.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.
Right so I guess the question is: do we make a new function mcissolve or add this as a flag to options. Maybe I'd be more inclined for a new function, but I don't have a strong preference