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

Change expected input of www/ir and mmm/ir tables #126

Open
sgwinder opened this issue Oct 23, 2020 · 0 comments
Open

Change expected input of www/ir and mmm/ir tables #126

sgwinder opened this issue Oct 23, 2020 · 0 comments
Labels
enhancement New feature or request Medium Priority

Comments

@sgwinder
Copy link
Member

Improve integration with viz-model by changing the column names that trails-viz expects in the www, wwwir, mmm, and mmmir tables.

wwwir

Current Column Proposed Column Reason
trail siteid Consistency
resp.ss weekly_viz
resp.ll days_ir_pc This column currently shows "days of data". In viz-model, I'm writing out a few relevant columns: days_ir is days of IR counts, days_pc is days of parking lot counts, and days_ir_pc is the sum. Though note that I am using that amount of data, but then extrapolating to get estimates of total visitation for the week
resplg drop We're not displaying this anywhere on the dashboard, so it's not necessary
d2p weekstart This one is ok... I've replaced it in viz-model with weekstart because this is more informative for me. I've been calculating d2p as weekstart + 2, so this would also be ok

mmmir

Current Column Proposed Column Reason
trail siteid
resp.ss monthly_viz
resp.ll weeks_ir monthly_viz is NOT like weekly_viz, in that we are simply adding up visitation from the weeks that we have estimates for. So weeks_ir is important for saying how many weeks went into this sum.
resplg drop

Social media tables

  • Possibly switch from "flickr", "twitter", "instag", and "wta" to PUD, TUD, IUD, and WUD

www

Current Column Proposed Column Reason
trail siteid
jjmm modeled_viz Or something like this, that's more interpretable than jjmm
jjmmlg drop
d2p weekstart
model I've been writing out the name of the model used to create the estimates
datecreated And the date that I did that round of modeling

mmm
Same suggestions as www, but keeping d2p as is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Medium Priority
Projects
None yet
Development

No branches or pull requests

1 participant