osuosl / streamwebs

Streamwebs.org website
http://streamwebs.readthedocs.io/
4 stars 1 forks source link

Histograms not working #660

Closed hertelc closed 6 years ago

hertelc commented 6 years ago

Tested via dev with 'jerryp/fix_water_points' branch.

The graph for Water Quality is looking better. But when I select the option to view the histogram for a point, the graph is not showing up.

Compare between dev and prod: http://streamwebs-dev.osuosl.org:8000/sites/wenaha-river/water/air_temperature/2016-09-28/ https://streamwebs.org/sites/wenaha-river/water/air_temperature/2016-09-28/

jerry-peng commented 6 years ago

@hertelc I've taken a look at the pages from both dev and prod. It looks like both Wenaha-river sites uses the old water quality graphing code. However, the one on prod has more water quality data sheets, thats why for the following 2 links for water quality graph, the dev has no graph and prod has some data on graph. http://streamwebs-dev.osuosl.org:8000/sites/wenaha-river/water/ https://streamwebs.org/sites/wenaha-river/water/ Because the dev does not have data on water quality, the histogram naturally does not have data. In normal usage, the user shouldn't be able to go to the histogram page, because the water quality graph does not have graphs, which means no data points to select on, which means no way to access that invalid histogram page unless specifically typing out the url that way.

I will test this theory with this new water quality graph code asap.

hertelc commented 6 years ago

@JerryPeng0112 unless you updated Dev, it likely has the old Water Quality code. I had Lance switch which branch was deployed many times today.

jerry-peng commented 6 years ago

@hertelc This is an issue on my end as well, will look into it asap. The theory above is not corerct.

jerry-peng commented 6 years ago

@hertelc I just pushed up a fix for this issue.

hertelc commented 6 years ago

@JerryPeng0112 this looks good, please merge. Please leave this issue open until we check it off on staging.

hertelc commented 6 years ago

Looks good in staging. Closing issue.