-
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
[wip] Tasks #3
Comments
The concept list can be added to concepticon, this will be better. Wait,
once I merged the data, so you can then check it and prepare a PR for
concepticon.
|
@MacyL, if the concept list can be interpreted as starostin, you can add starostin as the normal concept list, and you can make one exception for the one divergence. This is better than adding it to concepticon. |
I would prefer not to add to concepticon, because it's basically the same concept list. I calculated the differences between the two, and there are about 5 concepts - 8 concepts (add the ones you mentioned in the PR). |
you didn't read my follow-up reply. Essentially, I proposed the same.
What you should do, however, is add the concepts with ID to the
`concepts` variable, which we use as lookup. That's where this needs to
happen.
|
The text was updated successfully, but these errors were encountered: