Skip to content

Remove debug print statements from tests and replace them with assertions #210

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.

Sign up for GitHub

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

Merged
janosh merged 8 commits into from
Jun 10, 2025

primitive_neighbor_list raise RuntimeError not AssertionError on no a…

f7d4693
Select commit
Loading
Failed to load commit list.
Merged

Remove debug print statements from tests and replace them with assertions #210

primitive_neighbor_list raise RuntimeError not AssertionError on no a…
f7d4693
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jun 10, 2025 in 1s

2.40% of diff hit (target 80.00%)

View this Pull Request on Codecov

2.40% of diff hit (target 80.00%)

Annotations

Check warning on line 235 in torch_sim/autobatching.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/autobatching.py#L235 

Added line #L235 was not covered by tests

Check warning on line 405 in torch_sim/autobatching.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/autobatching.py#L405 

Added line #L405 was not covered by tests

Check warning on line 1029 in torch_sim/autobatching.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/autobatching.py#L1022-L1029 

Added lines #L1022 - L1029 were not covered by tests

Check warning on line 987 in torch_sim/math.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/math.py#L987 

Added line #L987 was not covered by tests

Check warning on line 179 in torch_sim/models/fairchem.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/fairchem.py#L178-L179 

Added lines #L178 - L179 were not covered by tests

Check warning on line 236 in torch_sim/models/interface.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/interface.py#L234-L236 

Added lines #L234 - L236 were not covered by tests

Check warning on line 272 in torch_sim/models/interface.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/interface.py#L265-L272 

Added lines #L265 - L272 were not covered by tests

Check warning on line 280 in torch_sim/models/interface.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/interface.py#L275-L280 

Added lines #L275 - L280 were not covered by tests

Check warning on line 288 in torch_sim/models/interface.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/interface.py#L283-L288 

Added lines #L283 - L288 were not covered by tests

Check warning on line 294 in torch_sim/models/interface.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/interface.py#L294 

Added line #L294 was not covered by tests

Check warning on line 298 in torch_sim/models/interface.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/interface.py#L297-L298 

Added lines #L297 - L298 were not covered by tests

Check warning on line 303 in torch_sim/models/interface.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/interface.py#L303 

Added line #L303 was not covered by tests

Check warning on line 308 in torch_sim/models/interface.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/interface.py#L308 

Added line #L308 was not covered by tests

Check warning on line 312 in torch_sim/models/interface.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/interface.py#L311-L312 

Added lines #L311 - L312 were not covered by tests

Check warning on line 317 in torch_sim/models/interface.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/interface.py#L317 

Added line #L317 was not covered by tests

Check warning on line 172 in torch_sim/models/mace.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/mace.py#L172 

Added line #L172 was not covered by tests

Check warning on line 648 in torch_sim/models/soft_sphere.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/models/soft_sphere.py#L645-L648 

Added lines #L645 - L648 were not covered by tests

Check warning on line 93 in torch_sim/neighbors.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/neighbors.py#L93 

Added line #L93 was not covered by tests

Check warning on line 112 in torch_sim/neighbors.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/neighbors.py#L111-L112 

Added lines #L111 - L112 were not covered by tests

Check warning on line 220 in torch_sim/neighbors.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/neighbors.py#L217-L220 

Added lines #L217 - L220 were not covered by tests

Check warning on line 1277 in torch_sim/optimizers.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/optimizers.py#L1276-L1277 

Added lines #L1276 - L1277 were not covered by tests

Check warning on line 1296 in torch_sim/optimizers.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/optimizers.py#L1295-L1296 

Added lines #L1295 - L1296 were not covered by tests

Check warning on line 1335 in torch_sim/optimizers.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/optimizers.py#L1334-L1335 

Added lines #L1334 - L1335 were not covered by tests

Check warning on line 1360 in torch_sim/optimizers.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/optimizers.py#L1359-L1360 

Added lines #L1359 - L1360 were not covered by tests

Check warning on line 1572 in torch_sim/optimizers.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

torch_sim/optimizers.py#L1571-L1572 

Added lines #L1571 - L1572 were not covered by tests
View more details on Codecov