geneontology / go-site

A collection of metadata, tools, and files associated with the Gene Ontology public web presence.
http://geneontology.org
BSD 3-Clause "New" or "Revised" License
46 stars 89 forks source link

Convert old annotation_qc.xml #282

Closed cmungall closed 7 years ago

cmungall commented 7 years ago

A while ago we agreed on xml for specifying our rules http://www.geneontology.org/quality_control/annotation_checks/annotation_qc.xml

These are translated to HTML placed on drupal, and a hyperlink provided for each rule: http://geneontology.org/page/annotation-quality-control#GO_AR:0000014

These URLs are provided in the owltools GAF checks

However, these have suffered some bitrot:

We should switch to the same system as https://github.com/geneontology/go-site/tree/master/metadata/gorefs

We can have a yaml structured blob inside human-readable markdown.

TODO

cmungall commented 7 years ago

Currently on a branch, can be viewed here for now:

https://github.com/geneontology/go-site/tree/qc-rules/metadata/rules

kltm commented 7 years ago

I think it's (more than) a little weird to have a YAML blob in the markdown--this is just the creation of another format. It would be also just much that harder to parse and control.

I think just straight YAML, with a large text/markdown blob as the final field would be more appropriate. YAML very much supports extended and easy-to-manage text fields.

kltm commented 7 years ago

E.g. > and | field controls.

cmungall commented 7 years ago

See also #287

cmungall commented 7 years ago

We need ASAP a quick view over all rules. I can do a first pass at the script.

cmungall commented 7 years ago

Some of the reasons for doing it this way:

On 6 Dec 2016, at 7:02, kltm wrote:

I think it's (more than) a little weird to have a YAML blob in the markdown--this is just the creation of another format. It would be also just much that harder to parse and control.

I think just straight YAML, with a large text/markdown blob as the final field would be more appropriate. YAML very much supports extended and easy-to-manage text fields.

-- You are receiving this because you were assigned. Reply to this email directly or view it on GitHub: https://github.com/geneontology/go-site/issues/282#issuecomment-265171958

kltm commented 7 years ago

You wrote it and it's here, so that counts for a lot. However:

It seems to be that the proposed format is at the same time not very good for neither computers (custom format, depends on an infrastructure) nor humans (embedded format, unknown classes, etc.).

cmungall commented 7 years ago

The html is a direct translation of the old qc xml, we should replace with md for ease of editing (but either works for now)

On 13 Dec 2016, at 20:16, kltm wrote:

You write it and it's here, so that counts for a lot. However:

  • is there a validator for this new format?
  • the jekyll connection is really about tying it to github at some level--I don't think you'd want to start bringing on ruby dependencies otherwise

It seems to be that the proposed format is at the same time not very good for neither computers (custom format, depends on an infrastructure) nor humans (embedded format, unknown classes, etc.).

-- You are receiving this because you were assigned. Reply to this email directly or view it on GitHub: https://github.com/geneontology/go-site/issues/282#issuecomment-266938510

cmungall commented 7 years ago

Will be easy to flip it all into md-in-yaml if we ever need to