Signify on synchweb pin too long/short due to pin detect

[DominicOram]

After #499 we will not be doing a grid scan at all if the pin is too/short long and so not registering a grid collection in ispyb. Previously when the pin was too long/short we signified this to the user by putting the grid collection in ispyb with an error state and a comment saying the error. Given that we're not registering a grid at all now we need a different way of informing the user.

Acceptance Criteria

• On pin too long/short there is a clear indication to the user in synchweb

[DominicOram]

The first place to try putting it is the rotation collection comment but this will require some GDA work. Ultimately we will skip the rotation as well so it might be better to put it in the robot load?

[DominicOram]

• Talk to synchweb if there are other ways to do this - define an experiment kind called error? Maybe @ndg63276 can help

[ndg63276]

The RobotAction table has a status field, which can be WARNING, might be the most suitable.

[DominicOram]

Hmm... That doesn't feel great because it's not really an issue with the robot load but a subsequent issue. Might be the simplest solution for now though

[DominicOram]

#996 is related

[DominicOram]

In the latest version we should get an error on the XRC entry. Confirm this and close the issue if we do.

[dperl-dls]

• this basically just means adding to the ispyb comment on pin too long/short
• we may get this for free?
• work here is to confirm it is working; if it isn't, do it

we think small

[DominicOram]

We do not currently get this for free as the warning comes out of pin tip detection and we don't add the callback until we send GRID_DETECT_AND_DO_GRIDSCAN later. To fix we probably want to move GRID_DETECT_AND_DO_GRIDSCAN to earlier