Skip to content

Add DOE Genesis Mission overview page#45

Merged
dragon-ai-agent merged 1 commit intomainfrom
genesis-mission-docs
Feb 21, 2026
Merged

Add DOE Genesis Mission overview page#45
dragon-ai-agent merged 1 commit intomainfrom
genesis-mission-docs

Conversation

@cmungall
Copy link
Member

Summary

  • Adds a curated narrative overview page at docs/subsets/genesis-mission.md covering all 19 Genesis Mission schema modules
  • Organized by domain: Energy & Grid, Nuclear, Earth Science, Computing, Industry, Materials & Physics, Bioprocessing & Labs
  • Each section links to auto-generated enum pages, includes example ontology mappings, and describes use cases
  • Coverage summary table and ontology overview (15 ontologies across 50 enums)
  • Adds "Subsets" nav section in mkdocs.yml

Test plan

  • mkdocs build completes with no errors on the new page
  • All internal links to ../elements/EnumName.md resolve correctly
  • Page accessible from site navigation under Subsets > DOE Genesis Mission

🤖 Generated with Claude Code

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>
Copilot AI review requested due to automatic review settings February 21, 2026 02:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.md with domain-organized enum links and example ontology mappings.
  • Updates mkdocs.yml navigation 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 |
Copy link

Copilot AI Feb 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
| SYNCHROTRON | ParticleAcceleratorType | NCIT:C48205 |

Copilot uses AI. Check for mistakes.
Comment on lines +9 to +11
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.
Copy link

Copilot AI Feb 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copilot uses AI. Check for mistakes.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot open a new pull request to apply changes based on this feedback

Comment on lines +17 to +21
| [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 |
Copy link

Copilot AI Feb 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copilot uses AI. Check for mistakes.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot open a new pull request to apply changes based on this feedback

Comment on lines +35 to +40
- [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
Copy link

Copilot AI Feb 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
- [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

Copilot uses AI. Check for mistakes.
@claude
Copy link
Contributor

claude bot commented Feb 21, 2026

Claude finished @cmungall's task —— View job


Reviewing PR: Add DOE Genesis Mission overview page

I've completed a comprehensive review of the PR. Here are my findings:

Todo List:

  • Read repository guidelines (CLAUDE.md)
  • Examine the added documentation file
  • Verify schema structure and enum references
  • Check mkdocs.yml changes
  • Provide comprehensive review feedback

⚠️ Critical Issues Found

1. Missing Enum Definitions

The documentation references 50 enums across 19 modules that do not exist in the current schema. I verified the entire src/valuesets/schema/ directory structure and found none of the Genesis Mission enums mentioned in the documentation:

Examples of missing enums:

  • GridComponentType, GridEnergyStorageType, GridManagementStrategyType
  • FusionConfinementType, FusionFuelType, FusionPlasmaParameterType
  • DataCenterCoolingType, SemiconductorMaterialType, QubitType
  • And 47 others listed in the documentation

Impact: All enum links (../elements/EnumName.md) will result in broken links when the documentation is built.

2. Non-Existent Schema Modules

The Coverage Summary table references 19 schema modules that don't exist:

  • grid, subsurface_energy, fusion, nuclear_cleanup, quantum, etc.

Actual schema structure includes modules like bio/, energy/nuclear/, computing/, materials_science/, etc., but no Genesis-specific modules.

3. Documentation Quality Issues

Content Accuracy

  • Line 134: References ParticleAcceleratorType in Computing section but it's not listed in Computing enums
  • Claims inconsistency: States "19 schema modules" and "each enum is mapped" but no supporting enums exist
  • Ontology mappings: Examples use valid CURIEs but reference non-existent enums

Structure & Navigation

  • mkdocs.yml changes are minimal and correct - properly adds Subsets navigation
  • File organization follows repository patterns - placed in docs/subsets/ as expected

4. Alignment with Repository Guidelines

Violates CLAUDE.md guidance:

  • Repository emphasizes "ALWAYS check IDs, NEVER guess them"
  • Documentation references non-existent schema elements
  • No corresponding schema changes to support the documentation

5. Code Quality Assessment

Positive Aspects

  • Well-structured markdown with clear sections and tables
  • Proper use of relative links (../elements/EnumName.md)
  • Professional documentation formatting
  • Logical domain organization
  • Valid ontology CURIE references

Issues

  • Functional: All enum links will be broken
  • Accuracy: Content doesn't match actual codebase
  • Maintenance: Creates documentation debt

Recommendations

Immediate Actions Required

  1. Either add the missing 50 enums to schema or remove references to non-existent enums
  2. Update schema module names to match actual file structure
  3. Fix the ParticleAcceleratorType reference on line 134
  4. Verify all ontology mappings against actual enum definitions

Alternative Approaches

  1. Create placeholder version: Document only existing energy/nuclear/computing enums from current schema
  2. Multi-PR approach: Create schema enums first, then add documentation
  3. Future roadmap: Convert to a "planned enums" document with clear disclaimers

Testing Recommendations

  • Run just site to verify documentation builds without broken links
  • Test navigation to ensure all referenced pages exist
  • Validate that ontology CURIEs resolve correctly

Security & Performance

No security concerns identified. Performance impact is minimal (static documentation).

Overall Assessment

While the documentation is well-written and professionally structured, it fundamentally references a schema that doesn't exist. This creates significant usability issues with broken links and misleading content. The PR should not be merged until the schema-documentation mismatch is resolved.


Copy link
Contributor

Copilot AI commented Feb 21, 2026

@cmungall I've opened a new pull request, #46, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Contributor

Copilot AI commented Feb 21, 2026

@cmungall I've opened a new pull request, #47, to work on those changes. Once the pull request is ready, I'll request review from you.

@dragon-ai-agent dragon-ai-agent merged commit 0e94e92 into main Feb 21, 2026
20 of 21 checks passed
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.

4 participants