-
Notifications
You must be signed in to change notification settings - Fork 2
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
Rename semantic
?
#581
Comments
In my opinion, it would be good to make a change just so that someone doesn't get an idea that this has something to do with the other ontology stuff from |
@Tara-Lakshmipathy, super, thanks for the quick feedback.
Overall there's no specific plan for the current classes to interact with |
I don't have a strong feeling. If you feel like changing it, I'm also fine with seeing a very different name. This being said, if you are concerned about people misunderstanding the class, |
@XzzX, does this usage sound OK from your CS perspective? Other ideas also welcome. |
I do not have an opinion about the naming. |
After the ontology discussion today, I have the feeling that it may be wise to rename
semantic
,Semantic
, andSemanticParent
, without any change to functionality. Currently I lean towardslexical
,Lexical
, andLexicalParent
, because the labels and paths all pertain to the scope of the semantic/lexical parent. https://en.wikipedia.org/wiki/Scope_(computer_science)It's a super easy change, but I want to get feedback because word choice can make things a lot easier or harder to understand. Feedback from all are welcome, but I'm going to assign the people I demand a comment from 😉
The text was updated successfully, but these errors were encountered: