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
_PY3_DATA_UPDATES uses win path if on win #777
Conversation
py3_data function does not work on windows as the item would never match a value in a windows path. _PY3_DATA_UPDATES now has a windows version if windows is used This is only a quick fix as having the list of theses data directories is not perfect.
Thanks for this. While it's not the final solution it's a helpful step in that direction. |
_PY3_DATA_UPDATES uses win path if on win
_PY3_DATA_UPDATES = ["chunkers/maxent_ne_chunker", | ||
"help/tagsets", | ||
"taggers/maxent_treebank_pos_tagger", | ||
"tokenizers/punkt"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
os.path.join() should join fragments correctly, taking care of '/' and ''.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that would be much cleaner, if you don't mind submitting a patch!
could you reopen the issue and assign it to me, so it doesn't get lost. |
cool! thanks |
_PY3_DATA_UPDATES uses win path if on win
py3_data function does not work on windows as the item would never match a value in a windows path.
_PY3_DATA_UPDATES now has a windows version if windows is used
This is only a quick fix as having the list of theses data directories is not perfect.