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

Specify trip columns by inclusion #67

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

allenmichael099
Copy link
Contributor

  • added cols_to_include_in_trips_table to config-defualt.yml
  • used dplyr:: select and any_of to select those columns before displaying for the interactive plot and for the table

In future changes, might want to unlist inferred labels somehow.

- added cols_to_include_in_trips_table to config-defualt.yml
- used dplyr:: select and any_of to select those columns before displaying for the interactive plot and for the table
@allenmichael099
Copy link
Contributor Author

This branch will help us avoid surprise issues when new fields get added to trips. Inferred labels, inferred trips, expectation to label, and expected_trip recently got added to the database. Inferred labels is another list column, but it does not get split into its sub-objects.

@asiripanich
Copy link
Owner

I'm not sure I like this as my e-mission-phone branch (rciti*) has none an embedded survey feature that contains non-standard responses. Let me test the master branch with a database from my deployment to see if I get any surprise issues.

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