1303 read smargon position in ispyb for rotation scans from hardware not GDA

[rtuck99]

Fixes #1303

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:

1. the DataCollection Position should now be populated by reading the smargon position instead of GDA
2. ispyb_extras no longer has the position attribute
3. Tests pass

[olliesilvester]

Also, we need to remember to do the associated GDA change when merging this

[rtuck99]

I know this changed happened a while ago, but why is DataCollectionPositionInfo distinct from DataCollectionInfo?

It's a separate sproc that you have to call to do the update, there is a special Position table which is referenced by a foreign-key in DataCollection