-
Notifications
You must be signed in to change notification settings - Fork 5
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
flowerType #50
Comments
Change term definition to: |
I am confused by the controlled vocabulary options... |
@fonturbel could you share your thoughts with us? What is confused? The ontology term: http://purl.obolibrary.org/obo/TO_0000882 ? |
Agree with definiton and vocabulary. Please, include mixed as an option in the controled vocabulary. |
Ok with me regarding the terms. |
Nothing to add |
Nothing to add. |
Nothin to add and agree with @zedomel |
Nothing to add |
I believe the name of the trait could be improved here as "flowerType" can be interpreted as other concepts. I suggest changing the name to "flowerOpening" or "flowerOpeningType" to be more specific to the user. Moreover, I agree with adding the "mixed cleistogamous and chasmogamous" option to the controlled vocabulary. |
I agree with @cepnunes "flowerType can be interpreted as other concepts" (e.g. corolla type) and I agree with @viniciusduartina. |
The point raised by @cepnunes is important to be considered. |
Nothing else to add. |
New label: |
cleistogamous
,chasmogamous
,http://purl.obolibrary.org/obo/TO_0000882
The text was updated successfully, but these errors were encountered: