search

DagsHub / fds

Fast Data Science, AKA fds, is a CLI for Data Scientists to version control data and code at once, by conveniently wrapping git and dvc
http://fastds.io
MIT License
382 stars 22 forks source link

fds init #97

Closed Chibuzo75 closed 3 years ago

Chibuzo75 commented 3 years ago

I successfully pip installed fastds, but anytime I did fds init, it always gives this import output after displaying some Traceback information: ImportError: cannot import name '__version__' from 'fds.__init__' < c:\users\<MyComputerName>\pneumonia-detection\.venv\lib\site-packages\fds\__init__.py>

guysmoilov commented 3 years ago

@Chibuzo75 strange! I would expect this to happen to use as well. Maybe it's a windows problem. Could you please share the Traceback information?

guysmoilov commented 3 years ago

And also maybe print the contents of c:\users\<MyComputerName>\pneumonia-detection\.venv\lib\site-packages\fds\__init__.py ?

guysmoilov commented 3 years ago

I think this error might be resolved if we move __version__ into its own file version.py. It's a bit strange to be importing things from fds.__init__, I don't think that's the intended use case for it. If we do this change, it will also require changing setup.py appropriately, since it takes that version number when deploying to PyPI.

Chibuzo75 commented 3 years ago

Hi Guy, Thank you for reaching out to me. Here are the trace back calls from usin both fds init and fds add.d: they gave the same output On Wednesday, 15 September 2021, 03:13:48 GMT-4, Guy Smoilovsky @.***> wrote:

@Chibuzo75 strange! I would expect this to happen to use as well. Maybe it's a windows problem. Could you please share the Traceback information?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

Chibuzo75 commented 3 years ago

Question Question

guysmoilov commented 3 years ago

@Chibuzo75 I hope this will be fixed in the next deployed version, I'll update here when it's deployed and you can test again

guysmoilov commented 3 years ago

@Chibuzo75 Hey, it would be great if you could test it with the newly deployed 0.5.0 version or newer, I'm interested to learn whether it fixed things.

I have to say I'm also not familiar with how conda runs things on windows (or in general) with runpy.py and how it affects things. Maybe a normal python installation would work? Can you ensure that the version of python you're getting is python 3?

Chibuzo75 commented 3 years ago

@guysmoilov Yes I do have python 3.8.2. The initial error is not displaying again. Now, it's most likely my local computer issue since it shows this when I type the fds init, fds

FYI: Dean Pleban has assisted me in completing the Pneumonia-Detection project but without the fds init.

guysmoilov commented 3 years ago

Thanks @deanp70 ! @Chibuzo75 well I would be happy to hear updates here if there are any, thanks!