Closed adrianusler closed 3 months ago
Attention: Patch coverage is 75.47170%
with 13 lines
in your changes are missing coverage. Please review.
Project coverage is 69.44%. Comparing base (
bb3bb2a
) to head (5790f8c
).
Files | Patch % | Lines |
---|---|---|
src/molara/Structure/io/importer_crystal.py | 75.00% | 11 Missing :warning: |
src/molara/Structure/structure.py | 75.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Codecov update on current improvements:
The recent updates to the codebase involve refining the structure and import functionalities within a scientific computing library, focusing on crystallography. Key changes include the removal of redundant methods, enhancements in object copying and atom manipulation, and improved file parsing flexibility. Tests have been expanded to ensure the new functionalities work as intended and to increase the overall code coverage, directly aligning with the project's goals of robustness and reliability.
Files | Changes |
---|---|
.../crystal.py |
Removed re import; Deleted from_poscar method |
.../importer_crystal.py |
Updated load method; Added parse_poscar method |
.../structure.py |
Added coords attribute; Modified copy method; Added center_of_mass property; Updated add_atom and remove_atom methods |
.../test_crystal.py |
Added and modified tests for POSCAR parsing and crystal object comparison; Implemented test_toggle_bonds |
.../test_molecule.py |
Added tests for toggle_bonds , copy , compute_collision , and center_coordinates ; Modified existing tests |
Objective | Addressed | Explanation |
---|---|---|
Increase test coverage of crystal.py, importer.py, importer_crystal.py, molecule.py, structure.py (#303) | ✅ |
🐇💻🌟
Changes abound, both big and small,
In code and tests, we've touched them all.
From crystals clear to molecules round,
New functionalities now abound.
So hop along on this code spree,
For better science, we all agree.
🌈🚀🐇
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Is this ready?
Is this ready?
I would say yes. I guess I won't continue working on this any time soon, so we might just as well merge it now.
Increase test coverage of crystal.py, importer.py, importer_crystal.py, molecule.py, structure.py
I need the [WIP] pull request to make sure that the tests are passing. My local pytest doesn't work at the moment.
Once finished, this closes #303.
Summary by CodeRabbit
center_of_mass
property for structures.