Open jamesdoyle21 opened 2 years ago
I disagree with this one... did chris specify this?
@mpiannucci this came from Jeff. What are your thoughts here? Perhaps a notification to let the user know they are comparing a model to a station prediction?
Yeha in theory kelly had mentioned being able to validate models against other models, so i see nothing wrong with validating against predictions IF and ONLY IF there are no actual measurements available for that given station
I agree with that. As long as we make note of it so the user isn't unknowingly assuming it's measured data. An orange notification in the same place we put the "error no data" should do it
There are often times when I want to compare model v pred. Likely because no other data exists. But in general, this tool should allow anything to be able to be compared to anything. The user needs to be aware of GIGO.
--
Chris Paternostro
https://tidesandcurrents.noaa.gov/
On Thu, Apr 14, 2022 at 1:51 PM James Doyle @.***> wrote:
I agree with that. As long as we make note of it so the user isn't unknowingly assuming it's measured data. An orange notification in the same place we put the "error no data" should do it
— Reply to this email directly, view it on GitHub https://github.com/asascience/ott-coops/issues/435#issuecomment-1099466738, or unsubscribe https://github.com/notifications/unsubscribe-auth/AO77FKP4LWYQF6K2MPDT65TVFBLHZANCNFSM5TLHRX5A . You are receiving this because you are subscribed to this thread.Message ID: @.***>
This issue came up when looking at Tide Predictions in the validation tool. I am getting no comparison options when I select tide predictions as the parameter. Is this a separate issue?
I also cannot use a station with only tide predictions as a comparison for water level - it spits an error.
@kvmartin001 do you have a test case for this issue?
@jamesdoyle21 No, in my PR, Tide Predictions was supposed to be stripped as an option in that field completely, so the case above is a bug. I could fix the bug with a one-line change, but I think we want to just allow Tide Predictions again, which would make the bug a moot point.
Tide predictions are enabled again (https://github.com/asascience/oceansmap-3/pull/356), so I moved this to Merged. Do we still want a notification?
Perhaps inline like this?
Yeah, I like that
If the user selects "Tide Predictions" as the parameter for the validation tool, is there a comparison model that is supposed to populate? Lets plan to discuss this in the meeting on Tuesday as well
Any water level
I am still getting no comparison models when I select Tide Predictions
Also, this ties into #434 but when the user uses the "Show Available Stations" button, the map is populated with all the stations that have water level listed, but will never have water level data available. And is the vast majority of these stations showing
I mean there's not really a way to filter based on that unless we get more information from the catalog. We only know what timesteps are available for the station, not which timesteps have what kind of data (right now)
OceansMap Bug Tracker line 63
This is obviously bad because we are validating a prediction with a prediction. The solution is to not allow the user to select these stations in the validation tool. If the user really wants to see both plotted on the same graph, they can do so in the obs drawer.
This relates to #434 because if we can differentiate these stations on the map, it should be easier to remove them from the validation tool.