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

Adding encoding #6

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

Conversation

SeanAmmirati
Copy link

No description provided.

@SeanAmmirati SeanAmmirati deleted the adding_encoding branch September 28, 2018 02:54
@SeanAmmirati SeanAmmirati restored the adding_encoding branch September 28, 2018 02:57
@SeanAmmirati SeanAmmirati reopened this Sep 28, 2018
@SeanAmmirati
Copy link
Author

Adding encoding specifications to force UTF-8 by default, and allows for additional encoding schemes to be specified in command line.

This was done because I initially tried to run this on a Windows machine but it would not run since the default encoding was CP-1252 rather than UTF-8, which resulted in an empty file for old Rmarkdown scripts (apparently, R-markdown scripts from other computers/the cloud can only be opened as UTF-8, not too sure if I'm right about this but this was the problem I was facing.) I changed the encoding to UTF-8 and it worked, so I figured I'd default this functionality and add it here in case other users face the same issue.

Not sure how (or if) ipython notebooks can even use another encoding scheme based on the code here, so I'm not sure how useful the encode option will be, but I added it for completeness.

Thanks!

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