search

google / riegeli

Riegeli/records is a file format for storing a sequence of string records, typically serialized protocol buffers.
Apache License 2.0
418 stars 53 forks source link

import error #8

Open Bingo20 opened 4 years ago

Bingo20 commented 4 years ago

Simply import riegeli gives me this error:

Traceback (most recent call last): File "test.py", line 22, in import riegeli File "build/bdist.linux-x86_64/egg/riegeli/init.py", line 21, in ImportError: cannot import name record_position

QrczakMK commented 4 years ago

How are you building the application?

The only mode currently supported by Riegeli is with bazel (and I admit that bazel’s way of importing other projects is currently awkward, requiring a WORKSPACE listing all transitive dependencies).

There is an effort to make a PIP-installable package with Riegeli but this is not done yet.

Bingo20 commented 4 years ago

@QrczakMK I changed to the python dir of riegeli-master, and run 'python setup.py build' and 'python setup.py install'. I want to use it in python

dayfine commented 4 years ago

any estimate on when there will be pip support?

QrczakMK commented 4 years ago

For building the package yourself, if you have TensorFlow installed (which should not really be a hard requirement but currently there is no option to skip TensorFlow-dependent stuff — this will change), then './configure' and 'python python/setup.py build' works (sdist and bdist too). The resulting package did not install for me though, complains about lack of setup.py inside…

I am sorry, this will need someone more knowledgeable to debug. I am not sure what is supposed to work and what not — I have no experience with Python packaging, only with regular Python coding, and during coronavirus times I am with this alone.

DaddyWesker commented 1 month ago

I'm facing the exact same issue. "'./configure' and 'python python/setup.py build' " doesn't work in my case. Anyone able to import riegeli?