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 missing info to short-term count docs #1077

Merged
merged 6 commits into from
Nov 4, 2024

Conversation

mkewins
Copy link
Contributor

@mkewins mkewins commented Sep 20, 2024

What this pull request accomplishes:

  • fills out missing information from category reference table, adds more info
  • per request, adds a table that shows the relationship between data loading systems, time availability, and where data lives

Issue(s) this solves:

n/a

What, in particular, needs to reviewed:

  • @radumas: added table
  • @jwrcoleman: can you give a quick once-over to the category reference table and make sure the info is correct?

What needs to be done by a sysadmin after this PR is merged

n/a

@mkewins mkewins changed the title Maddy/short term count docs update add missing info to short-term count docs Sep 20, 2024
this document is now redundant with the main README.md file
@radumas radumas linked an issue Sep 20, 2024 that may be closed by this pull request
add some context about the "cal" data dictionary
## Where can I find what data?

!['where-to-find-data'](../img/where-to-find-adhoc-count-data.png)

Copy link
Member

Choose a reason for hiding this comment

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

Can add a note here that the traffic_ tables have traffic_ stripped from them when they are transferred to the bigdata traffic schema. These tables are:

arterydata
cnt_det
countinfo
countinfomics
det
arc_link
category
node

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@radumas I updated the traffic_ to traffic. so I'm not sure this note is necessary anymore. I think it might introduce more confusion than clarify things?

Copy link
Member

Choose a reason for hiding this comment

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

I was thinking more in terms of identifying the tables that don't have a prefix (atr_ or tmc_) which are being actively updated, but maybe this leads into a broader discussion of cleaning up the traffic schema.

Copy link
Contributor

@jwrcoleman jwrcoleman left a comment

Choose a reason for hiding this comment

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

The category table looks good

@mkewins mkewins requested a review from radumas October 4, 2024 19:16
@radumas radumas merged commit cb08452 into master Nov 4, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Short term counting program documentation needs an overhaul
4 participants