-
Notifications
You must be signed in to change notification settings - Fork 23
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
Rework modular file structure #1592
Comments
@jannahastings answered to this via e-mail:
|
We try this out with a mini-ontology first, to see whether protege errors occur and if they can be fixed. |
Google table for removing classes to original modules |
How to move a class:
Workflow:
|
We should update the wiki article on the OEO structure: https://github.com/OpenEnergyPlatform/ontology/wiki/modules-of-the-oeo Maybe it would be also nice to have the structure diagram in README.md (without oeo-composed and without the blue colour of the new modules). |
@nelekoehler can you take care of the documentation and as well the README please? Here is the updated figure.: |
Yes, I can do that. |
I close this now as we will be releasing the new structure in a few minutes. |
Description of the issue
We decided on a modular structure for the OEO to allow the usage of single modules that focus on a sub-topic of the domain and to avoid merge conflicts.
However, our shared-module, which was initially created for the interaction of the modules, grew a lot and became the largest module of oeo recently. This counteracts the original idea of the modular structure.
Therefore, the file structure should be rethought to keep the advantages of the modules.
Ideas of solution
One idea is a new module, that contains only axioms across modules, called
oeo-shared-axioms
for now.I also included possible future modules as examples in the drawing: one for sectors and one for the composition of existing classes.
A restructuring like this might call for a version 2.0
Workflow checklist
I am aware that
The text was updated successfully, but these errors were encountered: