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

autotest: run relaxation tasks in parallel #573

Merged
merged 1 commit into from
Oct 15, 2021

Conversation

shazj99
Copy link
Contributor

@shazj99 shazj99 commented Oct 14, 2021

Currently, relaxation tasks are submitted and executed serially which is time consuming. This patch changes it to parallel using multiprocessing pool.

Change-Id: I9ef3f65729cc2e2f553ea7ac038e77d219ad4712

Change-Id: I9ef3f65729cc2e2f553ea7ac038e77d219ad4712
@codecov-commenter
Copy link

codecov-commenter commented Oct 14, 2021

Codecov Report

Merging #573 (2f1eff8) into devel (cb67774) will decrease coverage by 0.02%.
The diff coverage is 8.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel     #573      +/-   ##
==========================================
- Coverage   33.02%   33.00%   -0.03%     
==========================================
  Files          86       86              
  Lines       14739    14754      +15     
==========================================
+ Hits         4868     4870       +2     
- Misses       9871     9884      +13     
Impacted Files Coverage Δ
dpgen/auto_test/common_equi.py 54.60% <8.33%> (-4.52%) ⬇️

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 cb67774...2f1eff8. Read the comment docs.

@wanghan-iapcm wanghan-iapcm merged commit bb1277d into deepmodeling:devel Oct 15, 2021
Cloudac7 pushed a commit to Cloudac7/dpgen that referenced this pull request Dec 1, 2021
Change-Id: I9ef3f65729cc2e2f553ea7ac038e77d219ad4712

Co-authored-by: Zhengju Sha <jenny@bytedance.com>
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