search

aqualinkorg / aqualink-app

Aqualink.org Ocean Monitoring Application
MIT License
33 stars 11 forks source link

Allow multiple sensor depths #941

Closed echaidemenos closed 7 months ago

echaidemenos commented 9 months ago

resolves https://github.com/aqualinkorg/aqualink-app/issues/940

Changes the condition of uniqueness on sources, adding depth. Shows multiple hobo loggers in the TEMPERATURE ANALYSIS chart.

image

echaidemenos commented 8 months ago

The chart may be too crowded now, maybe we should split them? Also the card becomes too crowded, for now i am only showing one hobo logger.

github-actions[bot] commented 8 months ago

Build succeeded and deployed at https://aqualink-app-941.surge.sh (hash 04dd795cf0e3c98afb6ac6e2bf448ea581e2cb7d deployed at 2023-12-06T14:00:45)

echaidemenos commented 8 months ago

hello @Caesarh97! We made a test deployment for this PR here and uploaded some random sample data, how does it look?

The upload file had this form:

Temp,depth,Date,Time
19.6109769768861,1,10/08/2023,19:53
22.6625268511672,5,10/08/2023,19:53
20.0840630383378,10,,10/08/2023,19:53
...

Does this work out for the issue here? It could also be 3 different upload files, the important part is to include the depth field