search

SIO-ODF / ctdcal

A CTD calibration package in use on US GO-SHIP cruises
BSD 3-Clause "New" or "Revised" License
9 stars 4 forks source link

Assess AUX sensor processors #57

Open Klankers opened 3 weeks ago

Klankers commented 3 weeks ago

GO-SHIP doesn't use xmiss, fluor, or turb sensors outside of their engineering units, often reported as just a voltage.

Confirm that the functions present in CTDCAL all still work as intended and they are adequately documented.

Klankers commented 2 weeks ago

Add sensors to convert and confirm that they can be converted. Coefficients are almost always stored in a .PDF, so the user would need to supply them (provide "defaults"?)

asx- commented 2 weeks ago

You could do optical recognition, there must be libraries for that in python, but you’d still want a manual check by the analyst.

On Mon, Jun 17, 2024 at 21:46 Aaron @.***> wrote:

Add sensors to convert and confirm that they can be converted. Coefficients are almost always stored in a .PDF, so the user would need to supply them (provide "defaults"?)

  • P02: Sensor ID # 13 - CTDFLUOR (FLF-Fluorometer)
  • OSNAP32: ECO-FLNTU addition to config
  • OSNAP32: TURB/FLUOR to science units

— Reply to this email directly, view it on GitHub https://github.com/SIO-ODF/ctdcal/issues/57#issuecomment-2174753197, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABB5NCRQOXBJU55RHS7W5QDZH6GONAVCNFSM6AAAAABJK3GDSSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZUG42TGMJZG4 . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Klankers commented 1 week ago 
def seapoint_fluor(volts, coefs, decimals=6):
   """
   _check_coefs(coefs, ["GainSetting", "Offset"])
   # TODO: actual calibration/conversion/something?
   # TODO: move this to different module? edge case since it's the only Seapoint

More context. Support for seapoint fluorometer as well.