kundajelab / basepairmodels

MIT License
16 stars 6 forks source link

Interpretation failing due to old deepLIFT version #12

Open suragnair opened 3 years ago

suragnair commented 3 years ago

This is the error: File "/home/users/surag/oak/miniconda3/envs/basepairmodels/lib/python3.6/site-packages/basepairmodels/cli/interpret.py", line 192, in data_func return [dinuc_shuffle(model_inputs[0], args.num_shuffles, rng)] + \ TypeError: dinuc_shuffle() takes from 1 to 2 positional arguments but 3 were given

Followed installation instructions as specified in the docs with fresh conda env. It seems to install version 0.6.10.0 of deepLIFT. That version takes in only two arguments.

https://github.com/kundajelab/deeplift/blob/d0d5f3606b96ea6803b9c8bcb6c3b8e1f4b99300/deeplift/dinuc_shuffle.py#L41

Resolved by pip instal --upgrade deeplift. Perhaps specify version deepLIFT 0.6.13 or above in requirements.txt?