Skip to content

Fix for multiple smartem grid registration - #887

Merged
d-j-hatton merged 6 commits into
mainfrom
smartem-multiple-grid-registrations-fix
Sep 16, 2026
Merged

d-j-hatton merged 6 commits into
mainfrom
smartem-multiple-grid-registrations-fix

Conversation

@d-j-hatton

Copy link
Copy Markdown
Contributor

Currently multiple smartem grids are registered for each EPU atlas. Some of these are as a result of duplication in the code, some are because the Atlas.dm is edited triggering another endpoint hit. This should resolve all of these

@codecov

codecov Bot commented Sep 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.86207% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.59%. Comparing base (8b02ba1) to head (e0fc681).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #887      +/-   ##
==========================================
- Coverage   55.10%   53.59%   -1.52%     
==========================================
  Files         102      103       +1     
  Lines       11367    12137     +770     
  Branches     1508     1683     +175     
==========================================
+ Hits         6264     6505     +241     
- Misses       4767     5264     +497     
- Partials      336      368      +32     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread src/murfey/client/contexts/atlas.py Outdated
@d-j-hatton
d-j-hatton merged commit 62d0b5b into main Sep 16, 2026
14 checks passed
@d-j-hatton
d-j-hatton deleted the smartem-multiple-grid-registrations-fix branch September 16, 2026 10:06
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