Skip to content

Added new classes & slots in coremeta4cat_reaction_ap.yaml - #118

Merged
mvoelken-hub merged 7 commits into
nfdi4cat:mainfrom
mvoelken-hub:Dev-Marc
Jul 13, 2026
Merged

Added new classes & slots in coremeta4cat_reaction_ap.yaml#118
mvoelken-hub merged 7 commits into
nfdi4cat:mainfrom
mvoelken-hub:Dev-Marc

Conversation

@mvoelken-hub

Copy link
Copy Markdown
Collaborator

Added CatalystTypes, PerformanceMeasures, ReactionTypes and Reactor-Metadata field.

@mvoelken-hub
mvoelken-hub merged commit 8dcce15 into nfdi4cat:main Jul 13, 2026
6 checks passed
HendrikBorgelt added a commit to HendrikBorgelt/CoreMeta4Cat that referenced this pull request Jul 14, 2026
…rtifacts

Rebasing onto upstream/main surfaced additional duplication beyond the
already-resolved coremeta4cat_reaction_ap.yaml conflict: PR nfdi4cat#118 also added
has_cathode/has_anode/has_cell_operating_mode/has_active_area/
has_faradaic_current and several catalyst-bed/stirrer geometry slots to
coremeta4cat_common.yaml that duplicate slots already defined locally in
coremeta4cat_reaction_ap.yaml (same concepts, same VOC4CAT terms in most
cases, different slot names), plus has_conversion/has_space_time_yield/
has_selectivity referencing nfdi4cat#118's Conversion/SpaceTimeYield/Selectivity
classes -- which aren't included in this PR (see the PR description's note
on keeping those for separate discussion). Removes the now-orphaned
duplicates so the schema resolves cleanly, and regenerates all derived
artifacts against the final merged state.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant