search

chrisquince / STRONG

Strain Resolution ON Graphs
MIT License
44 stars 9 forks source link

Desman missing from conda dependencies #72

Open chrisquince opened 4 years ago

chrisquince commented 4 years ago

Desman is not on our list of conda dependencies - I tried installing it through conda though and it failed perhaps we need to create our own Desman conda package?

Sebastien-Raguideau commented 4 years ago

Ok, that's quite the overlook. We have it as a submodule, so we could just refer to it and add it's dependencies?

chrisquince commented 4 years ago

Hmmm... well yes we would need people to install it then. It is not recommended to mix conda and native installs. But as I said my conda desman install failed. We could try and get the maintainer of that package to fix it.

Sebastien-Raguideau commented 4 years ago

Does Desman needs compiling? Otherwise, install is just make it availlable in the path + having all its dependencies?

snurk commented 4 years ago

Since its your package that is likely to be changed why is it bad to have a submodule dependency?

snurk commented 4 years ago

I wouldn't want to rely on some other maintainer for DESMAN