OceanGlidersCommunity / Oxygen_SOP

Oxygen Standard Operating Procedure (SOP)
https://oceangliderscommunity.github.io/Oxygen_SOP/README.html
Other
6 stars 21 forks source link

Update pre deployment calibration procedure section #213

Closed tomhull closed 2 years ago

tomhull commented 2 years ago

Rewrite of the pre-deployment calibration steps and various smaller fixes. Should close #144

@patricialg one bit I don't understand and I think still needs review (I can open a issue if needed) is the bit about calibration in the ballast tank using a sonde. Surely this doesn't help because now you need to calibrate the sonde as well?

github-actions[bot] commented 2 years ago

This pull request is being automatically built with GitHub Actions and Netlify. To see the status of your deployment, click below.

🔍 Git commit SHA: d6999f747fee5a5dadfe8647196f12d896643244 ✅ Deployment Preview URL: https://623b1daa3ce93f35f218c8dc--oxygensop.netlify.app

patricialg commented 2 years ago

HI @tomhull , Yo are right about rewriting that. We used the sonde because we were calibrating it also because it was going the next day to the sea with the glider for the deployment (so it's not really necessary in all cases or you can decide to calibrate it another time). It also help to check if something is too wrong with glider or sonde sensors because you have both measuring at the same time ;) What do you think? Cheers, Patry

soerenthomsen commented 2 years ago

I agree with Tom in regards of the sonde in the tank and think it should not be a general recommendation.

soerenthomsen commented 2 years ago

@tomhull and @patricialg can you both coordinate to finish and merge this PR?

patricialg commented 2 years ago

I'm happy with the changes Tom did :)

soerenthomsen commented 2 years ago

Did we agree on the "the sonde in the tank" question? @tomhull @patricialg

patricialg commented 2 years ago

It was removed from the text since we all agreed that it's not something to consider as general recomendation. I checked and I couldn't find it in the new version. I'll check in cae appears in another place..

soerenthomsen commented 2 years ago

Ok perfect! Glad to see that we are closing a lot now! Soon we can submit :)