-
Notifications
You must be signed in to change notification settings - Fork 214
Fix flat_classification_report. #67
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
base: master
Are you sure you want to change the base?
Fix flat_classification_report. #67
Conversation
This is good, thank you for doing it. I hope it gets merged. |
Thanks! |
Is it possible to merge this PR and release a new version? @kmike ? |
Apparently it's not @costika1234, unfortunately I've lost commit access to this repo :( I still have pypi access though. It's a matter of putting some work to copy the repo somewhere. |
I see, @kmike. Do you happen to know the people who have access to this repo? If not, then I would advise moving this repo somewhere else so that you can deploy a new version containing this fix (and subsequent ones, too). |
I needed this change and the change from #69 for my code to work, so, seeing that the package is not updated anymore, I created a new fork with your fix and the fix from #69. If anyone needs both, you can start using an updated version of this package by running this: |
@MeMartijn Thanks for addressing this. However, I'm still getting the same error |
@patrickquick I'm sorry to hear it doesn't work for you still! I needed this package for a single project and never touched it again, but you can still contribute to my fork that had the initial fix. That way, you would help people that run into your issue too. Here's a link: https://github.com/MeMartijn/updated-sklearn-crfsuite |
If you're running it in a Jupyter notebook you may need to restart not only the kernel but the whole notebook server. |
Uh oh!
There was an error while loading. Please reload this page.