Skip to content

improve CLI performance#2696

Merged
wanghan-iapcm merged 5 commits intodeepmodeling:develfrom
njzjz:better-cli
Jul 26, 2023
Merged

improve CLI performance#2696
wanghan-iapcm merged 5 commits intodeepmodeling:develfrom
njzjz:better-cli

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Jul 25, 2023

Fix #2627.

Move the CLI method to a separate module to avoid the slow import of TensorFlow. The help message will be printed more quickly.

njzjz added 2 commits July 25, 2023 16:13
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz linked an issue Jul 25, 2023 that may be closed by this pull request
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -1.60% ⚠️

Comparison is base (b0b2b73) 78.72% compared to head (1c09eba) 77.13%.

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #2696      +/-   ##
==========================================
- Coverage   78.72%   77.13%   -1.60%     
==========================================
  Files         239      205      -34     
  Lines       25549    18878    -6671     
  Branches     1517        0    -1517     
==========================================
- Hits        20114    14561    -5553     
+ Misses       5045     4317     -728     
+ Partials      390        0     -390     
Files Changed Coverage Δ
deepmd/entrypoints/main.py 76.74% <100.00%> (-15.39%) ⬇️

... and 109 files with indirect coverage changes

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

njzjz added 2 commits July 25, 2023 16:43
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@wanghan-iapcm wanghan-iapcm merged commit 2f7d012 into deepmodeling:devel Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proposal for better cli entrypoints performance

2 participants