search

worldveil / dejavu

Audio fingerprinting and recognition in Python
MIT License
6.33k stars 1.43k forks source link

Does this package supports Python 3 #300

Closed valentindush closed 7 months ago

valentindush commented 8 months ago

I had troubles using it.. because it uses python 2

busterbeam commented 8 months ago

❓maybe you can provide a specific error?

tgrushka commented 7 months ago

It seems the current PyPI version does not. A print "... with no parentheses is certainly not Python 3 syntax.

pip install PyDejavu

Successfully installed PyDejavu-0.1.3

python -c 'from dejavu import Dejavu'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "./.venv/lib/python3.11/site-packages/dejavu/__init__.py", line 61
    print "%s already fingerprinted, continuing..." % filename
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: Missing parentheses in call to 'print'. Did you mean print(...)?
tgrushka commented 7 months ago

Install from GitHub according to INSTALLATION.md:

pip install https://github.com/worldveil/dejavu/zipball/master

is broken as well as it's requiring an outdated version of numpy.

busterbeam commented 7 months ago

🤦‍♂️️ @tgrushka PyDejavu-Rollong is the correct PyPI repository the one you used was someone else's fork of this repository that is 9 years out of date.

but sadly you are right about numpy as the way of so many applications now days, they get containerized because developers use libraries that are either really old (due to improper development processes) or still in alpha (thinking there is some benefit)