Skip to content
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 seed to md tutorial and ipynb #710

Merged
merged 3 commits into from
May 22, 2024
Merged

Conversation

brookwander
Copy link
Collaborator

adding random seed so there exists a frame set at the completion on adsorbate enumeration. This is because there are so few sites to accommodate running on CPU that sometimes there are no frame sets at the end of enumeration. In actual use, many more configs should be used.

@brookwander brookwander requested a review from mshuaibii May 22, 2024 13:36
Copy link

codecov bot commented May 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

@mshuaibii mshuaibii added this pull request to the merge queue May 22, 2024
Merged via the queue into main with commit 27ac2f3 May 22, 2024
5 checks passed
@mshuaibii mshuaibii deleted the setting-seed-for-ts-tutorial branch May 22, 2024 15:09
levineds pushed a commit that referenced this pull request Jul 11, 2024
* adding seed to md tutorial and ipynb

* removing tag to skip execution

* Revert "removing tag to skip execution"

This reverts commit 71766e3.
misko pushed a commit that referenced this pull request Jan 17, 2025
* adding seed to md tutorial and ipynb

* removing tag to skip execution

* Revert "removing tag to skip execution"

This reverts commit 71766e3.

Former-commit-id: ea202868ca14310344770a597409c3b1b5412c33
beomseok-kang pushed a commit to beomseok-kang/fairchem that referenced this pull request Jan 27, 2025
* adding seed to md tutorial and ipynb

* removing tag to skip execution

* Revert "removing tag to skip execution"

This reverts commit 039f50b [formerly 010365128c448070df563595b3d111d58a19fcaf].

Former-commit-id: 3af0bfc86d3bcf3c847dd229cd093b519c34e615
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants