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

Steps to get taxon exclusions from relation graph #2160

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cmungall
Copy link
Member

@cmungall cmungall commented Nov 9, 2021

This addresses the relation graph strategy outlined in #2137

@matentzn
Copy link
Contributor

Is this ready to merge? what are the associated steps, i.e. whats next?

@cmungall
Copy link
Member Author

Let's keep this unmerged for now until we have reached more clarity on #2137. If we decide to go a pure TBox strategy then we can close this.

However, I suspect at the least the graph strategy will be a highly useful adjunct to a TBox strategy. In which case we should tidy this PR:

  • decide on appropriate file names for products
  • which directories they should live in
  • how it should be integrated into the main flow

I think for these steps it should be someone more familiar with current ODK idioms that does this work

@matentzn matentzn marked this pull request as draft November 12, 2021 17:12
@github-actions
Copy link

This PR has not seen any activity in the past month; if nobody comments or reviews it in the next week, the PR editor will be allowed to proceed with merging without explicit approval, should they wish to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants