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

point to specific files for updates #79

Merged
merged 8 commits into from
Nov 29, 2022

Conversation

cnell-usgs
Copy link
Member

Designating the file vue reads in from s3 to avoid unintended updates from re-running the pipeline locally before ready to deploy.

This should create the ready-to-release update for the 2022 water year.

Copy link
Collaborator

@hcorson-dosch-usgs hcorson-dosch-usgs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to avoid having the site update before we're ready for it to, but I think the change to the filename should be reflected in 3_visualize.yml -- see my note about the different targets and one of the comments.

self.d3.csv("https://labs.waterdata.usgs.gov/visualizations/data/gw-conditions-site-coords.csv", this.d3.autotype),
self.d3.csv("https://labs.waterdata.usgs.gov/visualizations/data/gw-conditions-daily-proportions.csv", this.d3.autotype),
self.d3.csv("https://labs.waterdata.usgs.gov/visualizations/data/gw-conditions-daily-proportions-wy22.csv", this.d3.autotype),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like a good change to make, but should there be a corresponding change to the filename in 3_visualize.yml? Are you making the edit manually right now? I see you make a copy of the file with the dates in the filename further down. Should that be used here instead? And should this comment be updated to note that the file will need to be manually updated in GWL.vue?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes you are absolutely correct. However, this puts the data in the dev s3 bucket which cannot be accessed through a url in the browser. I did this manually as a band-aid to get the correct data showing for the last WY. We need to refine this piece, also to consider how we will set up to be able to change the dates to regenerate each month.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could redirect this to the public bucket instead of the dev as a solution. Definitely don't need to keep both. Also should consider how long we should store these updates once we have a routine down

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, sounds good. I'll make an issue with these notes, and then this is good to merge

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see #80

Copy link
Collaborator

@hcorson-dosch-usgs hcorson-dosch-usgs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to go - added issue #80 documenting how we'd like to refine the s3 approach.

@hcorson-dosch-usgs
Copy link
Collaborator

hcorson-dosch-usgs commented Nov 10, 2022

A potential issue - I just started working w/i this branch to make the visual for social media and noticed a couple of odd things:

  1. After the animation finishes, it immediate switches to this view:
    image

But that doesn't seem to reflect the starting conditions, if I play and immediately pause:
image

  1. Water levels in Florida and Hawaii are static and at 'very low' from October- January and then again April-end of September:
    image

@cnell-usgs
Copy link
Member Author

1 is a known issue with the first draw. I haven't been able to get to it yet https://github.com/USGS-VIZLAB/gw-conditions/issues/48The issue is that the svg that loads with the map initially represents 1 day of data, that is seemingly tied to the prior year's data. It must be that somewhere in the pipeline the map is created but is not linked to the updating data, or that we only created this once. This is set up like this because we wanted the map to appear right away when the page loads, and requiring the data to load first to be able to draw the map all in D3 was slow. Now, it loads a pre-defined svg and then when the data loads, binds it to the sites. This could be related to 2 also because if there were changes in sites between years, sites from last year may be loading and never getting data assigned, and remaining static on the page.

2 is suspicious and may be related to 1 if NA values are defaulting to the first draw state? also possible this is related to the different pcodes in FL.

Will look into this before moving forward.

@hcorson-dosch-usgs hcorson-dosch-usgs merged commit 8f831ed into DOI-USGS:main Nov 29, 2022
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

Successfully merging this pull request may close these issues.

2 participants