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

crfsuite-stdin: command not found #4

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

Conversation

MousaMohammed
Copy link

@MousaMohammed MousaMohammed commented Oct 6, 2020

No description provided.

@arne-cl
Copy link
Owner

arne-cl commented Oct 7, 2020

Dear @MousaMohammed,
thank you for your pull request!

I can't use it as is, since there are too many merge conflicts.
Can you split this into smaller PRs, i.e. one for updating nltk, one for improving heuristics etc?

(I will need to add / adapt tests to make this work and update the documentation
so users will know what the differences are.)

@MousaMohammed
Copy link
Author

Dear @MousaMohammed,
thank you for your pull request!

I can't use it as is, since there are too many merge conflicts.
Can you split this into smaller PRs, i.e. one for updating nltk, one for improving heuristics etc?

(I will need to add / adapt tests to make this work and update the documentation
so users will know what the differences are.)
Can you please write me your email, I want to ask you about something?

@arne-cl
Copy link
Owner

arne-cl commented Oct 12, 2020

Dear @MousaMohammed,
sure, my address is on my Github page.

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.

3 participants