AMLab-Amsterdam / lie_learn

Computations involving Lie groups and harmonic analysis
MIT License
190 stars 47 forks source link

J matrices do not download with setup.py develop #10

Closed bkmi closed 4 years ago

bkmi commented 5 years ago

Both python setup.py develop and pip install -e . do not download the J matrices file from google drive. Not terrible for someone who wants to open this repo in develop mode, but worth mentioning.

I will fix it later if I need it.

mys007 commented 4 years ago

Indeed, the installation seems to be broken because instead of downloading actual J_dense_0-278.npy, only the HTML page corresponding to the following notice is downloaded:

Google Drive can't scan this file for viruses.

J_dense_0-278.npy (221M) is too large for Google to scan for viruses. Would you still like to download this file?

This is then demonstrated in the code with OSError: Failed to interpret file '....../lib/python3.6/site-packages/lie_learn/representations/SO3/pinchon_hoggan/J_dense_0-278.npy' as a pickle

mariogeiger commented 4 years ago

Now the file is hosted on github