-
Notifications
You must be signed in to change notification settings - Fork 0
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
non-ASCII strings #38
Comments
you bet, where are you seeing these? |
I'm getting a whole slew of warnings when I check the package. I believe its the open response entries in |
got it. if the error messages are still handy to you, can you paste them in? And, are you using 2021 data now, or still only 2019? |
2019 only. This is what I get when I run an R CMD CHECK
|
Thank you!!! ok I see right away that we need to get rid of d_purpose_other |
also lol Boyfriend<80><99>s holiday work party and networking |
Liz: can you bring over tbi19 object from metc.tbi.helper and use that instead? |
Are you referring to https://github.com/Metropolitan-Council/metc.tbi.helper/blob/main/data/tbi_tables.rda? |
lol no because i hadn't pushed my commits yet! really winning today: https://github.com/Metropolitan-Council/metc.tbi.helper/blob/main/data/tbi19.rda |
Does this follow the same formatting as |
yes! but also note: the dictionary is in the object too, so you might need to replace calls to tbi_dict with tbi19$dictionary |
I believe this is now fixed with [ab3cbc7] over at metc.tbi.helper (Metropolitan-Council/metc.tbi.helper@ab3cbc7) |
Low-priority issue, but it would be nice if we could remove/replace non-ASCII characters
See potential solution https://stackoverflow.com/a/37254905
The text was updated successfully, but these errors were encountered: