-
Notifications
You must be signed in to change notification settings - Fork 51
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
added freshwater stream and biome #1477
Conversation
won't build becasue blank node owl:someValuesFrom obsolete alga
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some questions - revisions look good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do not merge something is wrong
Thanks. I saw the thread in EnvO editors slack too. I'll try to reorient myself now and possibly open a new PR. |
@pbuttigieg @cmungall I think the PR is OK now. No more blending of axioms between stream water, freshwater stream, and freshwater stream biome. |
rewrote the PR. will request new review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ID issue has been addressed, thanks!
Add equiv axiom for biome
leaving has part fresheater stream in for now
for DOSDP compliance
created 'freshwater stream ecosystem' and followed biome.yaml pattern |
can't build release because