Skip to content

Add python interface #1

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

Conversation

lucastheis
Copy link

Hi @kazuho, I've implemented a Python interface and some unit tests for your range coder. I've tested it with Python 2.7, 3.4, and on Linux and Mac. Let me know what you think.

@jsiloto
Copy link

jsiloto commented Aug 22, 2019

@lucastheis I can't use this lib. Could you help me?

$ pip install -e rangecoder
Obtaining file:///tf/rangecoder
Installing collected packages: range-coder
  Found existing installation: range-coder 1.0
    Can't uninstall 'range-coder'. No files were found to uninstall.
  Running setup.py develop for range-coder
Successfully installed range-coder
WARNING: You are using pip version 19.1.1, however version 19.2.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
from range_coder import RangeEncoder, RangeDecoder
from range_coder import prob_to_cum_freq, cum_freq_to_prob

ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-17-3f5fb52f5f83> in <module>
----> 1 from range_coder import RangeEncoder, RangeDecoder
      2 from range_coder import prob_to_cum_freq, cum_freq_to_prob

ModuleNotFoundError: No module named 'range_coder'

@lucastheis
Copy link
Author

Sorry, I do not know why your Python cannot find the range coder. Maybe you have multiple versions of Python installed and added the range coder to the wrong one?

@lucastheis lucastheis force-pushed the master branch 3 times, most recently from 8ffa226 to 1e914d8 Compare October 3, 2019 20:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants