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

add harvey #198

Merged
merged 1 commit into from
Sep 5, 2017
Merged

add harvey #198

merged 1 commit into from
Sep 5, 2017

Conversation

wdwatkins
Copy link
Collaborator

No description provided.

Copy link
Member

@aappling-usgs aappling-usgs left a comment

Choose a reason for hiding this comment

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

the text looks fine to me. my repo copy is way out of date, so can i just trust you that the app works with the new data? (or maybe @jiwalker-usgs will also test?)

webPropertyId: UA-78530187-7
internalWebPropertyId: '157530047'
viewName: All Web Site Data
timezone: America/Chicago
Copy link
Member

Choose a reason for hiding this comment

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

what do we do with this timezone field? looks like they're all America/Chicago?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think this gets used for the aggregated time of day plots. There are a few (NWIS I think) that are other tzs.

Copy link
Member

Choose a reason for hiding this comment

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

OK, thanks. i guess CDT makes sense for Harvey/Houston regardless

The VizLab Hurricane Harvey Water Footprint (VizLabHarvey) application demonstrates
Hurricane Harvey's approach to the U.S. Gulf coast on August 25-31, 2017. This
visualization shows the hurricane's impact on precipitation and streamflow.
projectContact: 'Jordan Walker'
Copy link
Member

Choose a reason for hiding this comment

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

sadness, but maybe we should phase jordan out now rather than trying to remember in 1 month?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good question, posted in slack

Copy link
Member

Choose a reason for hiding this comment

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

i saw that. it's OK with me if we proceed with Jordan listed - he can stay in charge until 9/30 =)

@aappling-usgs aappling-usgs merged commit ec13b29 into USGS-VIZLAB:master Sep 5, 2017
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