Skip to content

V3.4.0#189

Merged
sbillinge merged 86 commits intomainfrom
v3.4.0
Mar 27, 2026
Merged

V3.4.0#189
sbillinge merged 86 commits intomainfrom
v3.4.0

Conversation

@sbillinge
Copy link
Copy Markdown
Contributor

No description provided.

stevenhua0320 and others added 30 commits February 19, 2026 13:47
build: deprecate getLastAtom and add a test
chore: deprecate read/writeStr, placeInLattice method
chore: deprecate private function in utils
chore: deprecate setLatPar and setLatBase method
chore: deprecate abcABG and readStr method
chore: deprecate methods in `symmetryutilities.py`
sbillinge and others added 22 commits March 6, 2026 14:27
…ies-1

chore: deprecate expansion functional utilities method.
chore: deprecate Atom class method
chore: change utility script to snake_case method.
fix: fix load_structure method with loading Path object.
chore: add readme.md for devutils files
@sbillinge
Copy link
Copy Markdown
Contributor Author

@stevenhua0320 please could you look at this PR? It seems to be failing pre-commit because of an undefined variable. Also, I saw it import bareAtomLabel or something like that which we may need to deprecate?

@stevenhua0320
Copy link
Copy Markdown
Contributor

@stevenhua0320 please could you look at this PR? It seems to be failing pre-commit because of an undefined variable. Also, I saw it import bareAtomLabel or something like that which we may need to deprecate?

It seems that because of my first deprecation of method assignUniqueLabels is accidentally done on the main branch at the very beginning, and at that time we don't want to cover this method again in v3.4.0 and now github identifies the difference, so it believes we delete the change on assignUniqueLabels method. So a proper fix I believe is to make the change for assignUniqueLabels also on v3.4.0 so that this method is consistent with the change we have done in main. For the bareAtomSymbol, I believe we have deprecated that method in v3.4.0, so the only change we need to make is to change the usage to bare_atom_symbol. I would make a PR shortly to branch v3.4.0

@sbillinge
Copy link
Copy Markdown
Contributor Author

Ok. Sounds good. Are you able to make a PR?

@stevenhua0320
Copy link
Copy Markdown
Contributor

Ok. Sounds good. Are you able to make a PR?

Have made that PR, see #190

…tion

chore: add assignUniqueLabels deprecation message
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.20%. Comparing base (39d41f0) to head (a05255f).
⚠️ Report is 87 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #189      +/-   ##
==========================================
+ Coverage   98.93%   99.20%   +0.26%     
==========================================
  Files          13       15       +2     
  Lines        1884     2507     +623     
==========================================
+ Hits         1864     2487     +623     
  Misses         20       20              
Files with missing lines Coverage Δ
tests/test_atom.py 98.96% <100.00%> (+1.09%) ⬆️
tests/test_expansion.py 100.00% <100.00%> (ø)
tests/test_lattice.py 99.57% <100.00%> (+0.07%) ⬆️
tests/test_loadstructure.py 98.07% <100.00%> (+0.57%) ⬆️
tests/test_p_cif.py 99.70% <100.00%> (+0.10%) ⬆️
tests/test_p_discus.py 98.85% <100.00%> (ø)
tests/test_p_pdffit.py 99.41% <100.00%> (ø)
tests/test_parsers.py 99.47% <100.00%> (ø)
tests/test_spacegroups.py 98.73% <100.00%> (+0.73%) ⬆️
tests/test_structure.py 99.80% <100.00%> (+0.02%) ⬆️
... and 3 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sbillinge sbillinge merged commit 41c51ac into main Mar 27, 2026
7 checks passed
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.

2 participants