edgi-govdata-archiving / EEW_SDWA_NJ

An open educational resource developed at the Environmental Data and Governance Initiative for interactive exploration of New Jersey's waters and water pollution.
https://eew-sdwa-nj.streamlit.app/
GNU General Public License v3.0
1 stars 0 forks source link

#1 ask: click on a place, get the water service provider #96

Closed Frijol closed 1 year ago

Frijol commented 1 year ago

This is touched on in other issues & existing work, but I wanted to give it its own issue because it was Marianne's number one ask.

ericnost commented 1 year ago

@Frijol in #97 I have four different ways of retrieving PSAs and PWSs for a place. This branch should not be merged, but I've drafted a pull request to have a conversation there. The idea is that we pick one and implement it in a separate development.

Here's some description and my own thoughts on advantages/disadvantages. Curious what you think.

First, keep in mind that these are just experiments in the "click on a place" part. There are no additional charts, pages (e.g. EJ, lead lines, etc.), and no default places. I assume those extras would be relatively easy to implement afterwards (though maybe not, especially the "default".) Some error handling is still necessary too. Please finally note the SBU server is currently unable to return relatively large chunks of data, so what we have here is a bit modified and hacked. It works temporarily for the purposes of demonstration. I'll email Gil about a server fix (this has happened before, should be a quick fix).

Ok, this took all day. I'm about out of time with this project. Unfortunately, I think we'll have to be quite strategic in what we are able to advance moving forward. I think this is the priority though.

ericnost commented 1 year ago

Just added:

ericnost commented 1 year ago

P.S. As you navigate the experiments, keep in mind that some of them use some of the same st.session_state variables, which can cause some bugginess. When in doubt, clear the cache and re-run. This shouldn't be an issue when live because only one of the pages will be live.

ericnost commented 1 year ago

This provides a solution to the zoom issues mentioned above! Just give a warning to the user that they should zoom in more.... (on the slippy and button selection options) https://github.com/ericnost/streamlit_test/pull/97/commits/143d912bedc2f8127cb2680b0a640e6b6ebfd489

ericnost commented 1 year ago