-
Notifications
You must be signed in to change notification settings - Fork 878
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
pytest-split
in CI
#2704
pytest-split
in CI
#2704
Conversation
add test_files/.pytest-split-durations
…in pymatgen/io/vasp/tests/test_sets.py.")
Looks like this takes the runtime for the main @shyuep @mkhorton Wanted to get your opinion on this before merging since it's a bigger-than-usual change to the main CI workflow. We could add the same splitting to the |
Sure this is fine. |
…moments' on /pymatgen/io/phonopy.py:51 @requires(Phonopy, "phonopy not installed!") def get_pmg_structure(phonopy_structure): ... lattice = phonopy_structure.cell frac_coords = phonopy_structure.scaled_positions symbols = phonopy_structure.symbols masses = phonopy_structure.masses > mms = phonopy_structure.magnetic_moments
4b42aaa
to
0538c7d
Compare
3e02c75
to
9af323a
Compare
Just noticed the coverage increased by 18% in this PR. Is this because a fifth of the test suite wasn't running before with the old splitting setup? include:
- pkg: pymatgen/analysis/chemenv pymatgen/analysis/elasticity pymatgen/analysis/magnetism
pkg_id: 1
- pkg: pymatgen/analysis --ignore=pymatgen/analysis/chemenv --ignore=pymatgen/analysis/elasticity --ignore=pymatgen/analysis/magnetism
pkg_id: 2
- pkg: pymatgen/electronic_structure pymatgen/symmetry pymatgen/command_line pymatgen/ext
pkg_id: 3
- pkg: pymatgen --ignore=pymatgen/analysis --ignore=pymatgen/electronic_structure --ignore=pymatgen/symmetry --ignore=pymatgen/ext --ignore=pymatgen/command_line
pkg_id: 4 |
https://github.com/jerry-git/pytest-split is a
pytest
plugin that splits the test suite into equally sized splits based on test execution time.