Skip to content

Conversation

@stephen-riggs
Copy link
Contributor

Three unrelated minor fixes which I've grouped together:

  • The instrument server needs the machine config to know the data substrings and record movies to grafana
  • The atlas_update call has been nacking as there is no return in the statement
  • Foil hole ID errored as it was retrieved after the db had been closed

@stephen-riggs stephen-riggs self-assigned this Jan 30, 2025
@codecov
Copy link

codecov bot commented Jan 30, 2025

Codecov Report

Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 27.30%. Comparing base (51fd6ad) to head (1744c3f).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #475      +/-   ##
==========================================
- Coverage   27.31%   27.30%   -0.01%     
==========================================
  Files          78       78              
  Lines       10159    10162       +3     
  Branches     1338     1338              
==========================================
  Hits         2775     2775              
- Misses       7288     7291       +3     
  Partials       96       96              

@stephen-riggs stephen-riggs merged commit ab15836 into main Jan 30, 2025
17 checks passed
@stephen-riggs stephen-riggs deleted the web-post-fixes branch January 30, 2025 11:07
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.

3 participants