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

Fix email_classify.R under windows #23

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

Conversation

jnjcc
Copy link

@jnjcc jnjcc commented Jul 29, 2014

email_classify.R failed on R 3.0.3, Win7 x64, with some error messages like:

Error in seq.default(which(text == "")[1] + 1, length(text), 1):
'from' cannot be NA, NaN or infinite

There were some similar errors in issue #4.

This patch removes the "open" parameter from file function, which happens to
solve the above error for me, although I don't understand why it worked yet.

I've tested this patch on R 3.1.1, Ubuntu 12.04, it produced the same result with or
without the "open" parameter for me.

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.

None yet

1 participant