-
Notifications
You must be signed in to change notification settings - Fork 504
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
[r2] fix seeds in se_a and se_atten (#3880) #3947
Conversation
Fix deepmodeling#3799. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> - **New Features** - Introduced flexibility in specifying seed values, allowing either an integer or a list of integers. - Enhanced seed parameter usage across various initialization methods and classes for more controlled randomization. - **Improvements** - Updated seed initialization logic to include additional computations and dynamic adjustments. - Enhanced documentation for parameters in multiple classes, providing clearer usage guidelines. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu> (cherry picked from commit 0c472d1) Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
WalkthroughWalkthroughThis update primarily addresses issues with seed handling in the Changes
Assessment against linked issues
Tip AI model upgrade
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## r2 #3947 +/- ##
=======================================
Coverage 81.50% 81.51%
=======================================
Files 342 342
Lines 33865 33882 +17
Branches 2872 2876 +4
=======================================
+ Hits 27601 27618 +17
Misses 5381 5381
Partials 883 883 ☔ View full report in Codecov by Sentry. |
Summary by CodeRabbit
Bug Fixes
self.seed
conditionally in descriptor modules.Tests
refe
,reff
, andrefv
with new reference values.test_model_ener
method for better accuracy.These changes ensure more reliable descriptor computations and improved test accuracy.
(cherry picked from commit 0c472d1)