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

model_devi_clean_traj support int type #583

Merged
merged 3 commits into from
Nov 5, 2021

Conversation

KZHIWEI
Copy link
Contributor

@KZHIWEI KZHIWEI commented Nov 2, 2021

if model_devi_clean_traj is integer, then the most recent n iterations will be retained, others will be removed.

@codecov-commenter
Copy link

codecov-commenter commented Nov 2, 2021

Codecov Report

Merging #583 (6822bc8) into devel (2fd6c7e) will decrease coverage by 0.00%.
The diff coverage is 45.45%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel     #583      +/-   ##
==========================================
- Coverage   33.13%   33.12%   -0.01%     
==========================================
  Files          86       86              
  Lines       14799    14809      +10     
==========================================
+ Hits         4903     4906       +3     
- Misses       9896     9903       +7     
Impacted Files Coverage Δ
dpgen/generator/run.py 65.73% <45.45%> (-0.08%) ⬇️
dpgen/dispatcher/Dispatcher.py 32.48% <0.00%> (-0.42%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2fd6c7e...6822bc8. Read the comment docs.

Copy link
Collaborator

@AnguseZhang AnguseZhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributions!
Please update README correspondingly.

@KZHIWEI
Copy link
Contributor Author

KZHIWEI commented Nov 4, 2021

Thank you for the review. I have update description of model_devi_clean_traj in the README.

@AnguseZhang AnguseZhang merged commit 5ce256c into deepmodeling:devel Nov 5, 2021
Cloudac7 pushed a commit to Cloudac7/dpgen that referenced this pull request Dec 1, 2021
* model_devi_clean_traj support buffer

* fix clean index

* update readme
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.

3 participants