geneontology / neo

noctua entity ontology
9 stars 2 forks source link

Fix split limit bug; small env addition for testing #71

Closed kltm closed 3 years ago

kltm commented 3 years ago

Small add while testing for #70; environmental variable take and usage in README.

kltm commented 3 years ago

@cmungall I'm pretty sure that there is a bug in there with how split is being used with LIMIT. Fixing that, it looks like target/neo-mgi.obo likely has the right contents.

https://perldoc.perl.org/functions/split, search "LIMIT".

It would be great to get a second set of eyes on these changes.

kltm commented 3 years ago

Forgive the editor churn...

cmungall commented 3 years ago

How is it only now I discover LIMIT?

Yes this change is good