-
Notifications
You must be signed in to change notification settings - Fork 25
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
Remove other schemas #123
Comments
As we discussed on the call, this is a big change, so we'll want feedback from as many people as possible before making this change. I suggest that we move the frange and genetic_code schemas in the the main schema and remove the SO schema altogether (I really doubt anybody is using them). We can implement each of these changes on separate branches and with separate pull requests. The main motivation is to make it possible when creating more than one chado instance to have these tables "go along" with the rest of the chado. A good example of why this is necessary is genetic_code: you may create multiple chado instances each with separate genetic_code requirements. Additionally, there could be some overlap in feature names between chado instances in frange, so that schema shouldn't be shared among chados. |
I approve this change since it simplifies things when you work with multiple Chado instances. Some additional notes on the impact of such a change:
(as far as I know...) |
It seems this is a duplicate of issue #114 |
Summarizing #114 here in preparation for it being closed as a duplicate.
|
Additionally, confirming that KnowPulse does not use any of the ancillary schemas and I support these being moved into the main chado schema. |
It was proposed in our Tripal/Chado discussion this morning to move the frange, SO and genetic_code tables into the base schema of Chado and no longer have them separate.
This also requires adjusting the functions and views that may use these tables.
The text was updated successfully, but these errors were encountered: