-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Installing dependencies: issue with molecule-tda #277
Comments
Thanks for the feedback and sorry for the issues. Installation is indeed still a big issue as I'm waiting for a couple of PRs to be merged (e.g., #2, #202). Currently, you need to go via the "refactor" branch on my fork for moltda: Lines 59 to 73 in 3a8ab66
(Unfortunately, I ran out of CI time this month and hence the actions are failing) |
Ah, my mistake! I misread the instructions. For some reason, I thought you meant your fork was a refactor of the original. Oops...! Thanks! |
In this case I should probably rewrite to make it clearer ;) Thx! |
In hindsight, you were pretty clear. I just am over-eager in forking everything and then forked |
Closed this prematurely. I did
|
So sorry for the issues! I'm pretty sure it is that you're missing cgal (https://anaconda.org/conda-forge/cgal). Really need to get those conda things merged ....! |
It's all good! That's part of the process :) Maybe one step closer? This is what I get after installing
|
Now it seems to be https://anaconda.org/anaconda/gmp Bought some CI time and will fix the actions later today when I'm in the office. Do you mind if I ask what OS you are using? (I used clean conda envs on a bunch of UNIX machines and I didn't need to do many extra steps, would be good to know why you have a harder time) |
Thanks! I still got the error below. I'm using the Windows Subsystem for Linux rather than "real" Linux, so that may be a root cause of the problem (it's Ubuntu 20.04.3 LTS on a Windows 11 machine; a clean Conda environment with Python 3.8). I'll try a typical Linux environment shortly so I'm not wasting your time with debugging. Hopefully that'll go smoother.
|
Nono, really good to know! I think WSL is common and I do not have access so I'm happy that you're the guinea pig! 🧪 |
it is a bit hacky, but if you run mofdscribe/.github/workflows/tests.yml Lines 60 to 61 in 1f2a88d
does this lead you anywhere? |
Success! The key was |
shall we close this issue? I think the installation is a bit simpler now, even though it still requires manually conda installs to have the full functionality |
Yes, close! |
reopening as @sauradeep93 reported issues on Mac. It is interesting that the conda-installed cgal seems not to work. |
Hey Kevin,
I'm trying to install
mofdscribe
and all the dependencies. I'm having some trouble installingmolecule-tda
. I cloned the repo andpip install .
'd it and got back the following. This is on WSL with Python 3.8. I'm not too familiar withpoetry
so am still trying to figure this one out.The text was updated successfully, but these errors were encountered: