Skip to content

skip UTs when we do not have parmed, ase, pymatgen#249

Merged
amcadmus merged 5 commits into
deepmodeling:develfrom
amcadmus:skip-tests
Feb 25, 2022
Merged

skip UTs when we do not have parmed, ase, pymatgen#249
amcadmus merged 5 commits into
deepmodeling:develfrom
amcadmus:skip-tests

Conversation

@amcadmus

Copy link
Copy Markdown
Member

No description provided.

@amcadmus amcadmus requested a review from njzjz February 24, 2022 00:34
@codecov-commenter

codecov-commenter commented Feb 24, 2022

Copy link
Copy Markdown

Codecov Report

Merging #249 (8719bd7) into devel (4866a2f) will not change coverage.
The diff coverage is n/a.

❗ Current head 8719bd7 differs from pull request most recent head 9789d6d. Consider uploading reports for the commit 9789d6d to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##            devel     #249   +/-   ##
=======================================
  Coverage   82.00%   82.00%           
=======================================
  Files          59       59           
  Lines        5124     5124           
=======================================
  Hits         4202     4202           
  Misses        922      922           

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 4866a2f...9789d6d. Read the comment docs.

Comment thread tests/test_amber_md.py Outdated
Comment thread tests/test_ase_traj.py Outdated
Comment thread tests/test_pymatgen_molecule.py Outdated
wanghan-iapcm and others added 3 commits February 24, 2022 15:54
Co-authored-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Co-authored-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Co-authored-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@wanghan-iapcm wanghan-iapcm requested a review from njzjz February 24, 2022 07:55
@amcadmus amcadmus merged commit b62f67a into deepmodeling:devel Feb 25, 2022
@amcadmus amcadmus deleted the skip-tests branch February 26, 2022 12:26
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.

4 participants