Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

#1024 Update store_in_ispyb to use micronsPerPixel #1025

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

dperl-dls
Copy link
Collaborator

@dperl-dls dperl-dls commented Dec 6, 2023

Fixes #1024

Updates the field we write to, corresponding to recent ispyb-api updates.

You might need to pip install --upgrade ipsyb or redeploy the venv for this to work locally.

To test:

  1. Run tests

Copy link

codecov bot commented Dec 6, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (60a9997) 93.68% compared to head (75451db) 93.50%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1025      +/-   ##
==========================================
- Coverage   93.68%   93.50%   -0.19%     
==========================================
  Files          56       56              
  Lines        2708     2708              
==========================================
- Hits         2537     2532       -5     
- Misses        171      176       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@noemifrisina noemifrisina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks okay

@dperl-dls dperl-dls merged commit 369e8da into main Dec 7, 2023
16 checks passed
@dperl-dls dperl-dls deleted the 1024_update_ispyb_microns_per_pixel_field branch December 7, 2023 15:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update to reflect ISPyB pixels per micron changes
2 participants