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

GO Rules Review/Update #670

Closed
29 of 30 tasks
dougli1sqrd opened this issue Jun 2, 2018 · 4 comments
Closed
29 of 30 tasks

GO Rules Review/Update #670

dougli1sqrd opened this issue Jun 2, 2018 · 4 comments

Comments

@dougli1sqrd
Copy link
Contributor

dougli1sqrd commented Jun 2, 2018

Overriding as super:

  1. Review status of the rules: (checked = reviewed)
  2. if something needs to be done, a ticket is created and added to the project GO QC/QA/Rules 1.1 Release (2018-10, Fall GOC)
  3. Check current GO rules against previous implementation Check current GO rules against previous implementation #712
  4. Make reports prettier

**** original

Using this google document https://docs.google.com/document/d/1bVX8_Mf5OIFoVT_efawIvYDHBzNG-61hVZBfFLdOSqY/edit?usp=sharing as a guide, let's update the written GO Rules in metadata/rules/ in go-site to reflect some of the changes in the document.

Let's also create more tickets as necessary, and place them in https://github.com/orgs/geneontology/projects/20

This particular ticket is primarily about editing the rules to conform better to the things in the google doc.

Lastly, I've added a new field, type. Go Rules will be divided into a few types:

  • filter: This rule applies to individual lines in a GAF, and will throw out the line on a failure of the rule.
  • repair: This rule applies also to individual lines in a GAF, and will attempt to map a rule failure into compliance.
  • report: This can apply individually or collectively as a SPARQL query.

Any failure of a rule for any type should always report the failure.

@dougli1sqrd
Copy link
Contributor Author

https://github.com/geneontology/go-site/tree/gorule_update-670

This is the branch I'm working in. If anyone makes GO RULE changes as per the doc above, let's put it here.

@dougli1sqrd dougli1sqrd moved this from To do to Done in GO Rule Fixup and Implementations Jul 2, 2018
@kltm kltm changed the title GO Rules Update GO Rules Update/Review Jul 25, 2018
@kltm kltm changed the title GO Rules Update/Review GO Rules Review/Update Jul 25, 2018
@dougli1sqrd
Copy link
Contributor Author

Pascale says that for rule 26: If IBA annotations were made a contributor that is not GO_Central, that is a "real" error in the annotation. (Rather than just wanting to get rid of paint annotations in mod files)

@dougli1sqrd
Copy link
Contributor Author

Should we say this is done?

@pgaudet
Copy link
Contributor

pgaudet commented Nov 6, 2018

OK added tickets for everything - I think.

@pgaudet pgaudet closed this as completed Nov 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

No branches or pull requests

2 participants