Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restore Gauge Correlation Comment / Info Text Field #604

Closed
3 tasks done
ngottlieb opened this issue Nov 6, 2024 · 9 comments
Closed
3 tasks done

Restore Gauge Correlation Comment / Info Text Field #604

ngottlieb opened this issue Nov 6, 2024 · 9 comments
Assignees

Comments

@ngottlieb
Copy link
Contributor

ngottlieb commented Nov 6, 2024

Correlations used to have a comment field that allowed users to write things like, "This gauge is 10 miles downstream, so expect actual flows in the reach to be delayed X hours" (per Tony Allred comments on Streamteam forum).

@tallowen
Copy link
Member

Added to the db / api - you should be able to add a comment field now.

@ngottlieb
Copy link
Contributor Author

@AWKevin @riversandcreeks this is up on beta now, including the restoration of the old comments. Please let me know your thoughts on the interface for both reading and updating them.

NOTE: gauge-related information has no "beta" environment. What you see here within the flow tab is production data. Any edits you make will be reflected on production. When I'm testing things here, I will typically make changes to relatively obscure rivers and reverse it when I'm done.

@AWKevin
Copy link

AWKevin commented Jan 6, 2025

I don't see any correlation comments on flow tabs on Beta. Maybe I just can't find a river with a gauge comment - but I think I am visiting pages that should have them.

I added a comment to this river but did not see it reflected on the page after I saved it and clicked refresh a few times. https://beta.americanwhitewater.org/content/River/view/river-detail/11100/flow

UI worked well in terms of location and process. I think "Correlation comment" is kind of jargon from our team that is not intuitive. I suggest we say something more explicit like: "Comment on the correlation between this gauge and this reach (Optional)." We could even go longer like : "Comment on the correlation between this gauge and this reach (Optional). This comment will appear below the graph on the flow tab. Comments might include location of the gauge relative to the reach, time delays, tributary inputs, or other factors that affect the accuracy of the gage for this reach."

@ngottlieb
Copy link
Contributor Author

@AWKevin can you clarify (from the Slack discussion) if you do want me to move the correlation comments below the legend or leave them where they are?

I'll add the help text you suggested to the form

@AWKevin
Copy link

AWKevin commented Jan 7, 2025

I think it should for sure be moved to below the graph legend (at a minimum). My open question was whether it should be moved out of the right column entirely and placed under the graph as it used to be. My sense is that some of these comments are lengthy enough that they would make the right column much longer than the graph and lead to a weird presentation on desktop. I am open to either approach.

Screenshot 2025-01-07 at 9 08 36 AM

@AWKevin
Copy link

AWKevin commented Jan 7, 2025

Screenshot 2025-01-07 at 5 47 02 PM

@ngottlieb
Copy link
Contributor Author

@AWKevin I updated the form with helper text and rearranged the correlation details so it sits on top of both the graph and the level legend — that made sense to me and @riversandcreeks agreed on Slack but curious if you disagree. It's up on beta

@AWKevin
Copy link

AWKevin commented Jan 9, 2025

I like it!

@ngottlieb
Copy link
Contributor Author

This is live on prod now. There is still a pending investigation into old, missing data via @riversandcreeks and @tallowen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants