Skip to content
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

Error in the design pattern during the release #97

Closed
LilyAndres opened this issue Jul 3, 2022 · 12 comments
Closed

Error in the design pattern during the release #97

LilyAndres opened this issue Jul 3, 2022 · 12 comments

Comments

@LilyAndres
Copy link
Collaborator

LilyAndres commented Jul 3, 2022

I'm trying to run the make file, the command is:

./run.sh make all_modules

And I get the following error:

dosdp-tools generate --table-format=csv --template=patterns/chemical_concentration.yaml --outfile=modules/chemical_concentration.tmp.owl --obo-prefixes=true --infile=templates/chemical_concentration.csv
make: dosdp-tools: Permission denied
make: *** [Makefile:114: modules/chemical_concentration.owl] Error 127
@LilyAndres
Copy link
Collaborator Author

@kaiiam have you ever come across this type of error? Surely is not just about changing permissions...

@kaiiam
Copy link
Collaborator

kaiiam commented Aug 4, 2022

This is probably a docker issue as I am able to run it on my computer. Perhaps something we should ask Nico for help with.

@LilyAndres
Copy link
Collaborator Author

@matentzn I'm having this error when I'm trying to run the modules, by any chance you know what could be happening?

@matentzn
Copy link
Contributor

The error (127) means, usually, that, that a file was not found. So my guess is, given

dosdp-tools generate --table-format=csv --template=patterns/chemical_concentration.yaml --outfile=modules/chemical_concentration.tmp.owl --obo-prefixes=true --infile=templates/chemical_concentration.csv

That either:

  • patterns/chemical_concentration.yaml or templates/chemical_concentration.csv does not exist or:
  • the modules directory for modules/chemical_concentration.tmp.owl does not exist.

As an aside: pretty ancient ODK repo setup, could be updated

@kaiiam
Copy link
Collaborator

kaiiam commented Aug 10, 2022

We think it's a docker issue as I was able to run it no problem on my computer, and lilly has also run it in the past. @LilyAndres perhaps you can just double check it by cloning the repo again and just running it to make sure it wasn't some silly file deletion issue on your branch?

As an aside: pretty ancient ODK repo setup, could be updated

We created CDNO with ODK about 2 years ago, I guess that makes it ancient now :P

@LilyAndres
Copy link
Collaborator Author

@kaiiam I just cloned the repo and I still have the issue. However, I don't remember:

  • modules/chemical_concentration.tmp.owl

we don't have this file.

Question, should the concentration have these kind of permits:
Screen Shot 2022-08-10 at 9 36 21 pm

I think is not reading the chemical_concentration.owl

@matentzn
Copy link
Contributor

Does the modules directory exist?

@LilyAndres
Copy link
Collaborator Author

@matentzn yes, modules directory exist
Screen Shot 2022-08-10 at 9 55 38 pm

@kaiiam
Copy link
Collaborator

kaiiam commented Aug 10, 2022

Code works on my computer but not @LilyAndres I think this must be a docker issue or something like that. Can you update docker?

@matentzn
Copy link
Contributor

I can recreate the issue: INCATools/ontology-development-kit#651

@LilyAndres
Copy link
Collaborator Author

@matentzn thanks a lot, I followed the changes you made in ENVO and it's working now :) I will close this issue now.

@kaiiam
Copy link
Collaborator

kaiiam commented Aug 11, 2022

Excelent thank you very much @matentzn!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants