Skip to content

Feature MBEP#346 Additional diagrams#10

Merged
Lurkerpas merged 4 commits intomainfrom
feature-mbep#346-additional-diagrams
Mar 10, 2026
Merged

Feature MBEP#346 Additional diagrams#10
Lurkerpas merged 4 commits intomainfrom
feature-mbep#346-additional-diagrams

Conversation

@Lurkerpas
Copy link
Contributor

No description provided.

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 support for passing Interface/Deployment View source file paths into the Mako template context so templates can generate and embed additional diagrams (e.g., logical architecture) during document generation.

Changes:

  • Extend TemplateInstantiator to carry interface_view_path / deployment_view_path and expose them to templates.
  • Update CLI to pass the IV/DV input paths into the TemplateInstantiator.
  • Update the ECSS static architecture template to invoke SpaceCreator to generate an Interface View diagram and embed it in the output.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
templateprocessor/templateinstantiator.py Adds IV/DV path fields and injects them into the template render context.
templateprocessor/cli.py Passes the CLI --iv/--dv arguments through to TemplateInstantiator.
data/ecss-template/ecss-e-st-40c_4_1_software_static_architecture.tmplt Generates a logical architecture PNG via SpaceCreator and conditionally embeds it in the generated document.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Lurkerpas and others added 2 commits March 3, 2026 15:52
…ure.tmplt

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Lurkerpas Lurkerpas merged commit 6a272dc into main Mar 10, 2026
1 check 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