-
Notifications
You must be signed in to change notification settings - Fork 188
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 a how-to for catalyst-compiling "Symmetry-invariant quantum machine learning force fields" #1222
Open
paul0403
wants to merge
15
commits into
master
Choose a base branch
from
eqnn_forcefield_qjit
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.
Open
Add a how-to for catalyst-compiling "Symmetry-invariant quantum machine learning force fields" #1222
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
21f509b
Add a how-to for catalyst-compiling "Symmetry-invariant quantum machi…
paul0403 f83f690
grammar; remove unused imported
paul0403 442e9d2
move `opt_init, ... = optimizers.adam(...)` into the original location
paul0403 bbd9221
black format
paul0403 bdc6f4c
Merge remote-tracking branch 'origin/master' into eqnn_forcefield_qjit
paul0403 710b99a
Merge remote-tracking branch 'origin/master' into eqnn_forcefield_qjit
paul0403 c66e485
add qjit to original demo
paul0403 c72a374
remove second qjit demo
paul0403 58898d4
Merge remote-tracking branch 'origin/master' into eqnn_forcefield_qjit
paul0403 2d9ef0c
apply PR suggestions
paul0403 b4d58e7
Merge remote-tracking branch 'origin/master' into eqnn_forcefield_qjit
paul0403 d7845e4
update date of last modification
paul0403 f6adfd5
Merge remote-tracking branch 'origin/master' into eqnn_forcefield_qjit
paul0403 23e072d
trigger CI
isaacdevlugt 1823aa8
try CI with smaller run
paul0403 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
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.
When changing to catalyst, it was discovered that
QubitUnitary
cannot be differentiated:To make the demo work, I had to manually change gradient method to finite difference. This causes significant performance degradation.
Possible paths forward: