Update ms.yml #1014
Update ms.yml #1014
Conversation
config/ms.yml
Outdated
| replacement: https://github.com/HUPO-PSI/psi-ms-CV/releases/download/v | ||
| tests: | ||
| - from: /4.1.186/ms.owl | ||
| to: https://github.com/HUPO-PSI/psi-ms-CV/releases/download/v4.1.86/ms.owl |
There was a problem hiding this comment.
The tests are failing because "4.1.186" != "4.1.86". I assume that's a typo.
config/ms.yml
Outdated
| @@ -4,7 +4,8 @@ idspace: MS | |||
| base_url: /obo/ms | |||
|
|
|||
| products: | |||
| - ms.owl: https://github.com/HUPO-PSI/psi-ms-CV/releases/latest/download/psi-ms.owl | |||
| - ms.owl: https://github.com/HUPO-PSI/psi-ms-CV/releases/latest/download/ms.owl | |||
| - psi-ms.owl: https://github.com/HUPO-PSI/psi-ms-CV/releases/latest/download/psi-ms.owl | |||
There was a problem hiding this comment.
These products must all be of the form "id_space.file_extension". The ID_SPACEs are governed by OBO, and the relevant one here is "MS"/"ms". You're not allowed to add a new ID_SPACE here with "psi-ms.owl".
|
Hi @mobiusklein. Thanks for the PR -- I know it can be daunting. I made some comments. Maybe look at examples in this repo for other ontologies that you're familiar with. If you get stuck, let me know. Can you confirm for me in some way that you have authority to make these changes? A note from @germa (the official contact listed here https://obofoundry.org/ontology/ms.html) would be one way to let me know. |
|
I made some, if not all the requested changes, and fixed the typo in the tests, but wanted to see if the tests would fail still. @germa hasn't been involved in the PSI-MS CV's maintenance for a long time. I've been a maintainer for over two years, and was the contact person for two previous issues here: |
|
Thanks. I should have checked the source repo for your name, but I was in a rush. It would be helpful to update the OBO Foundry metadata for MR with the current contact (Principle 11): https://github.com/OBOFoundry/OBOFoundry.github.io/blob/master/ontology/ms.md |
This is an attempt to solve HUPO-PSI/psi-ms-CV#259 without rewriting how our OWL files are generated.
It keeps the names the same, but should now resolve versioned pURLs.
I am unable to setup a test VM at the moment, so I couldn't test how the blind
prefix: /rule would interact withproducts. From running thetranslate_yaml.pyscript, it didn't seem likeproductsproduced a rule in the generated.htaccessfile. If this would introduce a conflict, they can be removed?