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

remove sgd gpi entry #953

Merged
merged 1 commit into from
Dec 12, 2018
Merged

remove sgd gpi entry #953

merged 1 commit into from
Dec 12, 2018

Conversation

kltm
Copy link
Member

@kltm kltm commented Dec 5, 2018

Remove confusing GPI entry as a workaround for geneontology/neo#36

Remove confusing GPI entry as a workaround for geneontology/neo#36
@kltm kltm changed the title remove gpi entry remove sgd gpi entry Dec 5, 2018
@kltm kltm requested a review from cmungall December 5, 2018 19:37
@kltm
Copy link
Member Author

kltm commented Dec 12, 2018

@dougli1sqrd If @cmungall is not available, could you confirm this has the desired effect?

@dougli1sqrd
Copy link
Contributor

If this passes the schema checker, then this is fine. The pipeline doesn't look at GPI files yet, so this seems all good to me.

@kltm
Copy link
Member Author

kltm commented Dec 12, 2018

Hmmm...maybe it's looked at in the NEO assembler. I'll merge and I guess we'll see.

@kltm kltm merged commit 9090fba into master Dec 12, 2018
@dougli1sqrd dougli1sqrd deleted the kltm-patch-15 branch December 12, 2018 21:22
@cmungall
Copy link
Member

This should have the desired affect, thanks.

https://build.berkeleybop.org/job/build-noctua-entity-ontology/
ran successfully

Connection is too slow to check the file here

We'd expect gene IDs to show up here:

$ curl -L -s http://purl.obolibrary.org/obo/go/noctua/neo.owl | grep SGD

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

Successfully merging this pull request may close these issues.

None yet

3 participants