Skip to content

bug fix runaway classes (shortcut relations) #2119 #2023#2195

Open
stap-m wants to merge 19 commits intodevfrom
feature-2119-bug-fix-oeo-shared-axioms-reopen
Open

bug fix runaway classes (shortcut relations) #2119 #2023#2195
stap-m wants to merge 19 commits intodevfrom
feature-2119-bug-fix-oeo-shared-axioms-reopen

Conversation

@stap-m
Copy link
Contributor

@stap-m stap-m commented Jan 21, 2026

Summary of the discussion

Moving the shortcut relations to oeo-shared caused some runaway classes in oeo-pysical/model/social/sector. To fix this the domain/range axioms have to go back to oeo-shaed-axioms.

closes #2119

Type of change (CHANGELOG.md)

Update

  • has information input (shortcut) (domain)
  • has information output (shortcut) (domain)
  • covers energy carrier (shortcut) (domain and range)
  • covers technology (shortcut) (domain)
  • based on sector division (domain and range)

Workflow checklist

Automation

Closes #

PR-Assignee

Reviewer

  • 🐙 Follow the Reviewer Guide
  • 🐙 Provided feedback and show sufficient appreciation for the work done

@stap-m stap-m added this to Issues Jan 21, 2026
@github-actions github-actions bot added oeo-shared changes the oeo-shared module oeo-shared-axioms labels Jan 21, 2026
@github-project-automation github-project-automation bot moved this to Review in progress in Issues Jan 21, 2026
@github-actions
Copy link

github-actions bot commented Jan 21, 2026

Changed axioms per ontology

Note: Changes in term tracker annotations are not displayed

oeo-import-edits.owl: 2 changes

Ontology comparison

Left

  • Ontology IRI: https://openenergyplatform.org/ontology/oeo/oeo-import-edits.owl
  • Version IRI: https://openenergyplatform.org/ontology/oeo/dev/oeo-import-edits.owl
  • Loaded from: file:/__w/ontology/ontology/./dev/src/ontology/edits/oeo-import-edits.owl

Right

  • Ontology IRI: https://openenergyplatform.org/ontology/oeo/oeo-import-edits.owl
  • Version IRI: https://openenergyplatform.org/ontology/oeo/dev/oeo-import-edits.owl
  • Loaded from: file:/__w/ontology/ontology/src/ontology/edits/oeo-import-edits.owl

Ontology imports

Ontology annotations

MENO_01004 https://raw.githubusercontent.com/stap-m/midlevel-energy-ontology/main/ontology/src/midlevel-energy.owl/MENO_01004

Removed

Added

OEO_00020003 https://openenergyplatform.org/ontology/oeo/OEO_00020003

Removed

OEO_00020103 https://openenergyplatform.org/ontology/oeo/OEO_00020103

Removed

oeo-shared.omn: 9 changes

Ontology comparison

Left

  • Ontology IRI: https://openenergyplatform.org/ontology/oeo/oeo-shared/
  • Version IRI: https://openenergyplatform.org/ontology/oeo/dev/oeo-shared.omn
  • Loaded from: file:/__w/ontology/ontology/./dev/src/ontology/edits/oeo-shared.omn

Right

  • Ontology IRI: https://openenergyplatform.org/ontology/oeo/oeo-shared/
  • Version IRI: https://openenergyplatform.org/ontology/oeo/dev/oeo-shared.omn
  • Loaded from: file:/__w/ontology/ontology/src/ontology/edits/oeo-shared.omn

Ontology imports

Ontology annotations

OEO_00000365 https://openenergyplatform.org/ontology/oeo/OEO_00000365

Removed

OEO_00000368 https://openenergyplatform.org/ontology/oeo/OEO_00000368

Removed

OEO_00020039 https://openenergyplatform.org/ontology/oeo/OEO_00020039

Removed

OEO_00020227 https://openenergyplatform.org/ontology/oeo/OEO_00020227

Removed

based on sector division https://openenergyplatform.org/ontology/oeo/OEO_00390079

Removed

covers energy carrier (shortcut) https://openenergyplatform.org/ontology/oeo/OEO_00020432

Removed

covers technology (shortcut) https://openenergyplatform.org/ontology/oeo/OEO_00020438

Removed

  • covers technology (shortcut) elucidation "This is a shortcut relation to relate 'scenatio bundle' to 'technology'. The original relation 'covers technology' relates 'study' (process) to 'technology'.
    The shortcut relation should be used to represents these statements:
    ('scenario bundle 'is about' some 'scenario study'`) and ('scenario study' 'covers technology' some 'technology')"

  • covers technology (shortcut) Domain OEO_00020227

Added

  • covers technology (shortcut) elucidation "This is a shortcut relation to relate 'scenario bundle' to 'technology'. The original relation 'covers technology' relates 'study' (process) to 'technology'.
    The shortcut relation should be used to represents these statements:
    ('scenario bundle 'is about' some 'scenario study'`) and ('scenario study' 'covers technology' some 'technology')"

has information input (shortcut) https://openenergyplatform.org/ontology/oeo/OEO_00020437

Removed

has information output (shortcut) https://openenergyplatform.org/ontology/oeo/OEO_00020436

Removed

oeo-shared-axioms.omn: 7 changes

Ontology comparison

Left

  • Ontology IRI: https://openenergyplatform.org/ontology/oeo/oeo-shared-axioms
  • Version IRI: https://openenergyplatform.org/ontology/oeo/dev/oeo-shared-axioms.omn
  • Loaded from: file:/__w/ontology/ontology/./dev/src/ontology/edits/oeo-shared-axioms.omn

Right

  • Ontology IRI: https://openenergyplatform.org/ontology/oeo/oeo-shared-axioms
  • Version IRI: https://openenergyplatform.org/ontology/oeo/dev/oeo-shared-axioms.omn
  • Loaded from: file:/__w/ontology/ontology/src/ontology/edits/oeo-shared-axioms.omn

Ontology imports

Ontology annotations

based on sector division https://openenergyplatform.org/ontology/oeo/OEO_00390079

Added

covers energy carrier (shortcut) https://openenergyplatform.org/ontology/oeo/OEO_00020432

Added

covers technology (shortcut) https://openenergyplatform.org/ontology/oeo/OEO_00020438

Added

has information input (shortcut) https://openenergyplatform.org/ontology/oeo/OEO_00020437

Added

has information output (shortcut) https://openenergyplatform.org/ontology/oeo/OEO_00020436

Added

oeo.omn: 0 changes

Ontologies are identical

@stap-m
Copy link
Contributor Author

stap-m commented Jan 22, 2026

There are also sum runaway classes in the other modules, but they seem to be related to the oeo:energy transformation topic. I'll open a different PR for that.
EDIT: also some sectors in oeo-sector 😨
EDIT: I decided to fix the runaway classes related to oeo:energy transformation here as well.

@stap-m stap-m self-assigned this Jan 22, 2026
@github-actions github-actions bot added oeo-model changes the oeo-model module oeo-physical changes the oeo-physical module oeo-sector labels Feb 3, 2026
@github-actions github-actions bot added the oeo.omn changes the oeo.omn -module label Feb 3, 2026
@stap-m
Copy link
Contributor Author

stap-m commented Feb 3, 2026

Ok, the hierarchy of the submodules seems to be clean again. Could someone please add a review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

oeo.omn changes the oeo.omn -module oeo-model changes the oeo-model module oeo-physical changes the oeo-physical module oeo-sector oeo-shared changes the oeo-shared module oeo-shared-axioms

Projects

Status: Review in progress

Development

Successfully merging this pull request may close these issues.

[bug] the shortcut relations live in the wrong module

1 participant