Skip to content

Fixes for SmartEM integration#777

Merged
d-j-hatton merged 3 commits intomainfrom
smartem-fixes
Apr 7, 2026
Merged

Fixes for SmartEM integration#777
d-j-hatton merged 3 commits intomainfrom
smartem-fixes

Conversation

@d-j-hatton
Copy link
Copy Markdown
Contributor

A few things that came up in testing:

  • Atlas storage folder needs to be set to the destination so it is visible from the server
  • A DCG tag needs to be used to identify the correct data collection group so that the SmartEM UUID for the grid can be retrieved

… to the data collection group registration where a smartem grid will also be registered

this commit is just the client side
@d-j-hatton d-j-hatton requested a review from stephen-riggs April 7, 2026 10:34
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

❌ Patch coverage is 11.11111% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.88%. Comparing base (ad5e512) to head (c9aa896).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #777      +/-   ##
==========================================
- Coverage   48.89%   48.88%   -0.01%     
==========================================
  Files          94       94              
  Lines       10008    10012       +4     
  Branches     1326     1327       +1     
==========================================
+ Hits         4893     4894       +1     
- Misses       4860     4863       +3     
  Partials      255      255              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@d-j-hatton d-j-hatton merged commit 49c52ad into main Apr 7, 2026
17 checks passed
@d-j-hatton d-j-hatton deleted the smartem-fixes branch April 7, 2026 14:13
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