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

Made some changes to ojo_parse_county and added ojo_list_counties #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

andrewjbe
Copy link
Contributor

Changes to ojo_parse_county():

  • squish argument is now consistent regardless of other args; if TRUE, there will be no spaces, and if FALSE, all spaces will remain. I also altered the tests slightly to reflect the changed behavior.
  • suffix argument is now a real thing; it adds the argument to the county names and respects all other args

New ojo_list_counties() function:

  • Returns all 77 county names in lowercase by default, as a tibble with one col "value" to match the other ojo_list_ functions.
  • Also includes case, squish, suffix args from ojo_parse_county(), so you can pass those arguments along easily to get a formatted list.
  • Also includes oscn_only arg; if TRUE, will filter the list to only include the 13 oscn counties.

@andrewjbe
Copy link
Contributor Author

I was creating a UCCS code lookup table for a different project, and I figured it would be handy to have the script I used in /inst here. I also included the resulting dataset of unique charge descriptions; let me know if there's a better spot for it.

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.

1 participant