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

Screen and collect not working #1360

Closed
DominicOram opened this issue May 7, 2024 · 4 comments · Fixed by #1368
Closed

Screen and collect not working #1360

DominicOram opened this issue May 7, 2024 · 4 comments · Fixed by #1368
Assignees
Labels
needed_for_release Issues that must be complete before the next release

Comments

@DominicOram
Copy link
Collaborator

DominicOram commented May 7, 2024

When we do a screen and collect the screening should first put into ispyb with the experiment kind of Characterization, this causes agamemnon to tell GDA to wait for the screening to finish before the next collection. The experiment kind is passed into Hyperion straight from GDA. It looks as though the latest Hyperion version is not passing this into ispyb e.g. https://ispyb.diamond.ac.uk/samples/sid/5536566 should be a screen + collect.

Acceptance Criteria

  • Hyperion correctly passes experiment kind into ispyb from GDA
  • Screen and collect works
  • There is a test that would have caught this
@DominicOram DominicOram moved this to Candidates for Next Sprint in Hyperion May 7, 2024
@DominicOram DominicOram moved this from Candidates for Next Sprint to Backlog in Hyperion May 7, 2024
@DominicOram DominicOram moved this from Backlog to Candidates for Next Sprint in Hyperion May 7, 2024
@DominicOram DominicOram moved this from Candidates for Next Sprint to Todo This Sprint in Hyperion May 7, 2024
@DominicOram DominicOram added the needed_for_release Issues that must be complete before the next release label May 7, 2024
@d-perl
Copy link
Contributor

d-perl commented May 8, 2024

If the ispyb_experiment_type parameter needs to be passed in from agamemnon long-term it should be moved out of the TemporaryIspybExtras and into some other param class

@DominicOram
Copy link
Collaborator Author

We think small

@d-perl
Copy link
Contributor

d-perl commented May 8, 2024

There is currently logic in the ISPyB callback that determines if a collection is characterisation, we should also use this value rather than the number of images, probably with an enum corresponding to the valid collection types

@d-perl d-perl self-assigned this May 9, 2024
@DominicOram DominicOram moved this from Todo This Sprint to In Progress in Hyperion May 9, 2024
d-perl added a commit that referenced this issue May 9, 2024
@d-perl
Copy link
Contributor

d-perl commented May 9, 2024

Turns out there was already both a mechanism for this and a test for it, the use of it just got lost in the new parameter generation of ispyb params.

@DominicOram DominicOram moved this from In Progress to Review in Hyperion May 9, 2024
d-perl added a commit that referenced this issue May 9, 2024
@DominicOram DominicOram moved this from Review to In Progress in Hyperion May 10, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Hyperion May 14, 2024
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this issue Aug 23, 2024
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this issue Aug 23, 2024
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this issue Aug 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needed_for_release Issues that must be complete before the next release
Projects
Archived in project
2 participants