Add DOE Genesis Mission overview page#45
Conversation
Curated narrative doc linking to all 19 Genesis-related schema modules, organized by domain (energy, nuclear, earth science, computing, industry, materials/physics, bioprocessing/labs) with example ontology mappings and a coverage summary table. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Adds a new documentation page intended to provide a narrative overview of “DOE Genesis Mission” value sets and exposes it in the MkDocs navigation.
Changes:
- Adds a new overview page at
docs/subsets/genesis-mission.mdwith domain-organized enum links and example ontology mappings. - Updates
mkdocs.ymlnavigation to include a new “Subsets” section pointing to the new page.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| mkdocs.yml | Adds a “Subsets” nav group and links to the new Genesis Mission page. |
| docs/subsets/genesis-mission.md | Introduces a curated overview page with coverage tables, enum links, and example mappings. |
| | SILICON | SemiconductorMaterialType | CHEBI:27573 | | ||
| | SILICON_CARBIDE | SemiconductorMaterialType | CHEBI:29390 | | ||
| | PHOTONIC | QubitType | CHEBI:30212 | | ||
| | SYNCHROTRON | ParticleAcceleratorType | NCIT:C48205 | |
There was a problem hiding this comment.
In the Computing example mappings, ParticleAcceleratorType is referenced even though it isn’t listed in the Computing enums and (more importantly) doesn’t exist in the current schema sources, so this row is likely a copy/paste error and will become a dead link/invalid example.
| | SYNCHROTRON | ParticleAcceleratorType | NCIT:C48205 | |
| The 19 schema modules below cover topics from grid modernization and fusion energy to | ||
| quantum computing and autonomous laboratories. Each enum is mapped to established | ||
| ontologies wherever possible, enabling integration with broader scientific data ecosystems. |
There was a problem hiding this comment.
The text states the overview covers “19 schema modules” and that each listed enum is mapped, but none of the referenced Genesis-specific enums (e.g., GridComponentType, FusionConfinementType, DataCenterCoolingType) exist anywhere in the current schema sources under src/valuesets/schema (or the merged docs schema), so these links won’t have corresponding generated pages and will render as dead links.
There was a problem hiding this comment.
@copilot open a new pull request to apply changes based on this feedback
| | [Energy and Grid](#energy-and-grid) | grid, subsurface_energy | 6 | OEO, ENVO, CHEBI | | ||
| | [Nuclear](#nuclear) | fusion, nuclear_cleanup, nuclear_forensics | 8 | CHEBI, NCIT, CHMO, ENVO | | ||
| | [Earth Science](#earth-science) | subsurface, hydrogeology, remote_sensing, water_resources | 12 | ENVO, PATO, CHEBI, OBI | | ||
| | [Computing](#computing) | data_centers, microelectronics, quantum | 6 | Brick, CHEBI, NCIT | | ||
| | [Industry](#industry) | manufacturing, construction, unconventional_resources | 7 | OBI, CHMO, ENVO, Brick | |
There was a problem hiding this comment.
The “Schema” names in the Coverage Summary (e.g., grid, subsurface_energy, nuclear_cleanup, remote_sensing, microelectronics, etc.) don’t correspond to any existing schema modules/files in src/valuesets/schema, so readers won’t be able to reconcile this page with the actual project structure. Consider updating this table to reflect the real module names, or adding the missing Genesis modules in the same PR.
There was a problem hiding this comment.
@copilot open a new pull request to apply changes based on this feedback
| - [GridComponentType](../elements/GridComponentType.md) -- transmission lines, transformers, substations, inverters | ||
| - [GridEnergyStorageType](../elements/GridEnergyStorageType.md) -- battery chemistries, pumped hydro, hydrogen storage | ||
| - [GridManagementStrategyType](../elements/GridManagementStrategyType.md) -- demand response, load balancing, virtual power plants | ||
| - [SubsurfaceEnergyResourceType](../elements/SubsurfaceEnergyResourceType.md) -- geothermal, oil, gas hydrate reservoirs | ||
| - [SubsurfaceStorageType](../elements/SubsurfaceStorageType.md) -- CCS, underground hydrogen, compressed air | ||
| - [ReservoirCharacterizationMethodType](../elements/ReservoirCharacterizationMethodType.md) -- well tests, seismic, core analysis |
There was a problem hiding this comment.
These enum links (GridComponentType, GridEnergyStorageType, GridManagementStrategyType, SubsurfaceEnergyResourceType, SubsurfaceStorageType, ReservoirCharacterizationMethodType) don’t have corresponding enum definitions in the current schema sources, so just gen-doc won’t generate the target docs/elements/*.md pages and the links will be broken.
| - [GridComponentType](../elements/GridComponentType.md) -- transmission lines, transformers, substations, inverters | |
| - [GridEnergyStorageType](../elements/GridEnergyStorageType.md) -- battery chemistries, pumped hydro, hydrogen storage | |
| - [GridManagementStrategyType](../elements/GridManagementStrategyType.md) -- demand response, load balancing, virtual power plants | |
| - [SubsurfaceEnergyResourceType](../elements/SubsurfaceEnergyResourceType.md) -- geothermal, oil, gas hydrate reservoirs | |
| - [SubsurfaceStorageType](../elements/SubsurfaceStorageType.md) -- CCS, underground hydrogen, compressed air | |
| - [ReservoirCharacterizationMethodType](../elements/ReservoirCharacterizationMethodType.md) -- well tests, seismic, core analysis | |
| - `GridComponentType` -- transmission lines, transformers, substations, inverters | |
| - `GridEnergyStorageType` -- battery chemistries, pumped hydro, hydrogen storage | |
| - `GridManagementStrategyType` -- demand response, load balancing, virtual power plants | |
| - `SubsurfaceEnergyResourceType` -- geothermal, oil, gas hydrate reservoirs | |
| - `SubsurfaceStorageType` -- CCS, underground hydrogen, compressed air | |
| - `ReservoirCharacterizationMethodType` -- well tests, seismic, core analysis |
|
Claude finished @cmungall's task —— View job Reviewing PR: Add DOE Genesis Mission overview pageI've completed a comprehensive review of the PR. Here are my findings: Todo List:
|
Summary
docs/subsets/genesis-mission.mdcovering all 19 Genesis Mission schema modulesmkdocs.ymlTest plan
mkdocs buildcompletes with no errors on the new page../elements/EnumName.mdresolve correctly🤖 Generated with Claude Code