search

ersilia-os / eos2r5a

Retrosynthetic accessibility score
Other
1 stars 5 forks source link

Clean UP & Dockerization of eos2r5a #4

Closed GemmaTuron closed 1 year ago

samuelmaina commented 1 year ago

Hello @GemmaTuron, I have opened a PR. Can you have a look.

GemmaTuron commented 1 year ago

Perfect, let's keep an eye on the Actions runs, the PR is now merged

GemmaTuron commented 1 year ago

Please check the DockerHub run

samuelmaina commented 1 year ago

@GemmaTuron , the build is failing at amd, with this error No such file or directory: '/root/eos/dest/eos2r5a/pack.py' , I looked at the PR last commit and saw that I had added pack.py in the root. I don't know what could have caused this.Any idea?

GemmaTuron commented 1 year ago

Hi @samuelmaina Seems the workflow runs on the pre-merged version of the repo, I've rerun it manually and the error is not the same any more, please check and also read this issue: https://github.com/ersilia-os/eos74bo/issues/20

samuelmaina commented 1 year ago

Hi @GemmaTuron , I have made a PR, you can review it.

miquelduranfrigola commented 1 year ago

Thanks @samuelmaina - I have merged the PR, let's see if it works: https://github.com/ersilia-os/eos2r5a/actions

In any case, since we have updated workflows, I will run them again.

samuelmaina commented 1 year ago

Thanks @miquelduranfrigola , I am watching the actions.

miquelduranfrigola commented 1 year ago

Hello @samuelmaina - I am now running the action with the new workflows. Let's see if it works. We will update you all about the new workflows today.

samuelmaina commented 1 year ago

The workflows are not failling .Thanks @miquelduranfrigola

miquelduranfrigola commented 1 year ago

@GemmaTuron - This model seems to be resolved. Please check

GemmaTuron commented 1 year ago

There seems to be an issue with the output of the model to csv? @samuelmaina can you please provide an update on this?