cioos-siooc / cioos-siooc/ocean-data-parser

IOS Conversion - moored currentmeter data issues

Open
#82 2 comments 0 reactions 1 assignee Claimed by @JessyBarrette View on GitHub
Dominant language
Jupyter Notebook
Stars
5
Forks
2
PR merge metrics
No merged PRs in 30d

Description

For IOS moored Current meter data, I’ve compared the netCDF file generated by ODPY and by the old ios_data_tranform, and would like to suggest a few items for discussion and potential modification:

- [ ] Varaible record_number is set as float in moored CTD file. The variable type should be consistent in moored CTD and moored current meter files.
To do: change the record number from int to float.

- [x] Adding long_name and standard_name to variable time as shown below:
double time(time) ;
time:_FillValue = NaN ;
time:long_name = "time" ;
time:standard_name = "time" ;
time:units = "seconds since 1970-01-01 00:00:00+0000" ;

- [ ] Missing variable instrument_depth in the ODPY version:
float instrument_depth ;
instrument_depth:_FillValue = NaNf ;
instrument_depth:long_name = "Instrument Depth" ;
instrument_depth:standard_name = "instrument_depth" ;
instrument_depth:units = "m" ;

- [ ] Variable names in the ODPY version are not BODC standard names.
In ODPY version old version standard name in the old version

| ios_name | BODC | standard_name|
|---|---|---|
|Speed\:East |LCEWEL01 |"eastward_sea_water_velocity"|
|Speed\:North | LCNSEL01 | "northward_sea_water_velocity"|
|Speed\:Up | LRZASP01 |"upward_sea_water_velocity"|
|Amplitude\:Beam1 | ISCMBMA1 |"signal_intensity_from_multibeam_.....”|
|Amplitude\:Beam2 | ISCMBMA2 |"signal_intensity_from_multibeam_.....”|
|Amplitude\:Beam3 | ISCMBMA3 |"signal_intensity_from_multibeam_.....”|
|Speed\:Sound\:1 | SVELCV01 |"speed_of_sound_in_sea_water"|
|Heading| HEADCM01 |"platform_orientation"|
|Pitch| PTCHEI01 |"platform_pitch"|
|Roll | ROLLEI01 |"platform_roll"|
|Speed\:Current | LCSAEL01 |"sea_water_speed"|

Question: should I add these variables to the dfo_ios_vocabulary.csv

- [ ] There are a few metadata under data section in the old version:
```
filename = "chat3_20220722_20230802_0075m_L1.cur" ;
country = "Canada" ;
deployment_mission_id = "2023-069" ;
event_number = "4" ;
profile = "2023-069-0004" ;
scientist = "Hannah C." ;
project = "BC-shelf 7" ;
agency = "IOS, Ocean Sciences Division, Sidney, BC" ;
platform = "John P. Tully" ;
instrument_type = "Aquadopp Current Meter" ;
instrument_model = "AQUADOPP" ;
instrument_serial_number = "AQUADOPP-17257" ;
instrument_depth = 75 ;
latitude = 54.52715 ;
longitude = -130.5415 ;
geographic_area = "Hecate-Strait " ;
```
Question: are these necessary?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.