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

Support forward files for simplify #1092

Merged
merged 3 commits into from
Dec 30, 2022

Conversation

HuangJiameng
Copy link
Collaborator

Support "user_forward_files" for simplify in the 02.fp step. Use abspath to create symlinks to avoid unexpected problems.

support "user_forward_files" for `simplify` in 02.fp step, use abspath to create symlink
@codecov-commenter
Copy link

codecov-commenter commented Dec 29, 2022

Codecov Report

Attention: Patch coverage is 33.33333% with 4 lines in your changes missing coverage. Please review.

Project coverage is 39.54%. Comparing base (a9d324b) to head (fa5eb16).
Report is 193 commits behind head on devel.

Files Patch % Lines
dpgen/simplify/simplify.py 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #1092      +/-   ##
==========================================
- Coverage   39.55%   39.54%   -0.01%     
==========================================
  Files          99       99              
  Lines       17981    17986       +5     
==========================================
+ Hits         7112     7113       +1     
- Misses      10869    10873       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

dpgen/simplify/simplify.py Outdated Show resolved Hide resolved
@AnguseZhang AnguseZhang merged commit 06bd270 into deepmodeling:devel Dec 30, 2022
@ysu-007
Copy link

ysu-007 commented Jan 3, 2023

Have you tested it? It is still not working for me.

@HuangJiameng
Copy link
Collaborator Author

Have you tested it? It is still not working for me.

img_v2_9b34c63e-779a-466d-92d8-4d22937a049g

@ysu-007
Copy link

ysu-007 commented Jan 3, 2023

Are the parameters in the machine.json right?
image

@HuangJiameng
Copy link
Collaborator Author

Are the parameters in the machine.json right? image

The correct parameter is "user_forward_files".

@ysu-007
Copy link

ysu-007 commented Jan 3, 2023

Are the parameters in the machine.json right? image

The correct parameter is "user_forward_files".

It works now, thanks

@HuangJiameng HuangJiameng deleted the simplify_uff branch January 3, 2023 07:47
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.

5 participants