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

PyPi package is not up to date #83

Closed
RianFuro opened this issue Feb 10, 2016 · 8 comments
Closed

PyPi package is not up to date #83

RianFuro opened this issue Feb 10, 2016 · 8 comments

Comments

@RianFuro
Copy link

Hi! I just compared your github codebase to the your PyPi package (installed via pip) and noticed that the code there seems to be out of date (even though it gives the same version number).

Would be cool to have an up-to-date version on pypi since there's a problem with ascii escape sequences in the pypi package that doesn't seem to exist with the github version :)

@sigmavirus24
Copy link
Member

Hi @RianFuro,

We're planning a large release but neither @dan-blanchard nor I have very much time for chardet. It isn't as simple as updating the package on PyPI because we have to pull together the last few pull requests, documentation, changelog entries, and testing before releasing it.

The version number being the same is purely because we do not change the version number here until we're ready to release the version.

@adamn
Copy link

adamn commented May 17, 2016

I would like a release as well. I got this traceback recently and the offending code looks alot cleaner now so is probably already fixed.

P.S. - I know the below code is embedded in Requests - I'll create a ticket for them to use the PyPI version of chardet once it's released :-)

builtins:MemoryError
........
File "/var/virtualenvs/centr/lib/python3.5/site-packages/requests/packages/chardet/__init__.py", line 30, in detect
File "/var/virtualenvs/centr/lib/python3.5/site-packages/requests/packages/chardet/universaldetector.py", line 128, in feed
File "/var/virtualenvs/centr/lib/python3.5/site-packages/requests/packages/chardet/charsetgroupprober.py", line 64, in feed
File "/var/virtualenvs/centr/lib/python3.5/site-packages/requests/packages/chardet/sbcharsetprober.py", line 72, in feed
File "/var/virtualenvs/centr/lib/python3.5/site-packages/requests/packages/chardet/charsetprober.py", line 57, in filter_without_english_letters
File "/var/virtualenvs/centr/lib/python3.5/re.py", line 182, in sub

@sigmavirus24
Copy link
Member

@adamn please don't create an issue for requests. I'm a core developer in both places. When we have a good version here, it's my intent to update it in requests. Thanks :)

@dan-blanchard
Copy link
Member

The main thing blocking a new release (for me) is that we still have failing tests. I would like to have them all passing before we push out a release.

There's also a major set of changes in this branch based off #52, an abandoned PR with loads of good changes (including the ability to retrain out models). I'm going to try to put in some time to finish that up this week.

@1e0ng
Copy link

1e0ng commented Jul 20, 2016

Any good news? anything I can help? 😄

@dan-blanchard
Copy link
Member

I've made some process on the branch I was talking about before, but it's still not 100%. The main issue is that I'm working on the script to retrain our language models, since the one in the abandoned PR are not quite right, and neither are the old ones we're using currently. I recently moved, so I've been busy with house stuff, but I'll try to snag some more time for this soon.

@dan-blanchard
Copy link
Member

I will try to cut a new release today. I know having some of the fixes we have in master would be better than waiting endlessly for the big one (#99) to be done.

@dan-blanchard
Copy link
Member

Fixed by #110.

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

No branches or pull requests

5 participants