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

1302 do not put position into ispyb for gridscans #1401

Merged
merged 5 commits into from
May 23, 2024

Conversation

rtuck99
Copy link
Contributor

@rtuck99 rtuck99 commented May 21, 2024

Fixes #1302

Link to dodal PR (if required): #N/A
(remember to update setup.cfg with the dodal commit tag if you need it for tests to pass!)

To test:

  • Tests should pass
  • Grid scans will no longer accept position in ispyb_extras

@rtuck99 rtuck99 requested review from d-perl and DominicOram May 21, 2024 13:02
@rtuck99 rtuck99 marked this pull request as ready for review May 21, 2024 13:04
Base automatically changed from 1276_use_new_params_in_callbacks to main May 21, 2024 16:27
@d-perl d-perl self-assigned this May 22, 2024
Copy link
Contributor

@d-perl d-perl left a comment

Choose a reason for hiding this comment

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

lgtm!

@rtuck99 rtuck99 merged commit 90721aa into main May 23, 2024
14 of 18 checks passed
@rtuck99 rtuck99 deleted the 1302_do_not_put_position_into_ispyb_for_gridscans branch May 23, 2024 08:37
olliesilvester pushed a commit to olliesilvester/mx-bluesky that referenced this pull request Aug 23, 2024
…/hyperion#1401)

* (DiamondLightSource/hyperion#1302) Remove position populate from ispyb callback code

* (DiamondLightSource/hyperion#1302) Remove position from external ispyb gridscan parameters

* (DiamondLightSource/hyperion#1302) fixes for system tests (WIP)

* (DiamondLightSource/hyperion#1302) Remove surplus JSON test file, add position test to rotation ispyb system test

* (DiamondLightSource/hyperion#1302) Make pyright happy
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.

Do not put position into ispyb for gridscans
2 participants