Closed kenko911 closed 2 months ago
This update primarily involves modifications to the matgl
library and associated tests. Key changes include the simplification of the collate_fn_pes
function by removing a condition related to magnetic moments, and the introduction of a new test fixture for MoS2 in molecular dynamics simulations. Additionally, enhancements were made to data loader tests and the generation of Clebsch-Gordan coefficients in the SO3 utilities.
File Path | Change Summary |
---|---|
.../graph/data.py |
Removed conditional return in collate_fn_pes for include_magmom . |
tests/conftest.py |
Added MoS2() fixture. |
tests/ext/test_ase.py |
Replaced MoS with MoS2 in test_molecular_dynamics . |
tests/graph/test_data.py |
Added test_mgl_dataloader_without_collate_fn . |
tests/utils/test_so3.py |
Added test case for generating Clebsch-Gordan coefficients without parity invariance. |
tests/ext/test_ase.py (1)
`118-128`: Changes to use `MoS2` in `test_molecular_dynamics` are aligned with PR objectives.Verification successful
--- The verification process has confirmed that the function `test_molecular_dynamics` uses the `MoS2` fixture as intended. The search results show that this function is defined in `tests/ext/test_ase.py` and correctly uses `MoS2` as a parameter, aligning with the changes described in the PR. **Changes to use `MoS2` in `test_molecular_dynamics` are aligned with PR objectives.** - Verified in `tests/ext/test_ase.py`.Scripts Executed
The following scripts were executed for the analysis: --- Script: ```shell #!/bin/bash # Description: Verify all function calls to `test_molecular_dynamics` use `MoS2`. # Test: Search for the function usage. Expect: Only occurrences of `MoS2`. rg --type python $'test_molecular_dynamics' ``` Length of output: 73 --- --- Script: ```shell #!/bin/bash # Description: Verify all function calls to `test_molecular_dynamics` use `MoS2`. # Test: Search for the function usage. Expect: Only occurrences of `MoS2`. rg --glob '*.py' 'test_molecular_dynamics' ``` Length of output: 99tests/graph/test_data.py (1)
`485-622`: Addition of `test_mgl_dataloader_without_collate_fn` enhances testing capabilities as intended. Ensure comprehensive test coverage for `MGLDataLoader` functionality.Verification successful
--- The search results from the `rg` command indicate that `MGLDataLoader` is extensively used in multiple test files (`tests/utils/test_training.py` and `tests/graph/test_data.py`). This suggests a broad coverage of the `MGLDataLoader` functionality across different scenarios within the test suite. The presence of multiple calls to `MGLDataLoader` in these files likely covers various configurations and use cases, which is essential for thorough testing of this component. Given this information, it appears that the test coverage for `MGLDataLoader` is comprehensive, addressing the initial concern raised in the review comment. --- **Comprehensive test coverage for `MGLDataLoader` confirmed.** - The usage of `MGLDataLoader` is well-represented in the test suite, particularly in `tests/utils/test_training.py` and `tests/graph/test_data.py`, which suggests thorough testing of its functionalities.Scripts Executed
The following scripts were executed for the analysis: --- Script: ```shell #!/bin/bash # Description: Verify comprehensive test coverage for `MGLDataLoader`. # Test: Search for tests related to `MGLDataLoader`. Expect: Coverage of all functionalities. rg --type python $'MGLDataLoader' ``` Length of output: 63 --- --- Script: ```shell #!/bin/bash # Description: Verify comprehensive test coverage for `MGLDataLoader`. # Test: Search for tests related to `MGLDataLoader`. Expect: Coverage of all functionalities. rg --type py 'MGLDataLoader' ``` Length of output: 2278
Summary
Included more united tests to improve code coverage
Checklist
ruff
.mypy
.duecredit
@due.dcite
decorators to reference relevant papers by DOI (example)Tip: Install
pre-commit
hooks to auto-check types and linting before every commit: