Fix code and documentation related to Processing Unit Application - #21
Fix code and documentation related to Processing Unit Application#21imdshr wants to merge 5 commits into
Conversation
| A ProcessingUnitApplication must exist: register a hardware platform | ||
| (`add_hardware_platform`) or, for VEOS/no-hardware, call `add_application_processing_unit`. | ||
| (`add_hardware_platform`) or, for VEOS/no-hardware, call `add_processing_unit_application`. |
There was a problem hiding this comment.
This is not correct. Registering a hardware platform has nothing to do with a processing unit application.
At least one processing unit application is mandatory for a ConfigurationDesk application regardless of whether a real-time application shall be created or a BSC.
Would be nice if this prerequisite would not be necessary, and that instead the create_application_process would automatically create a processing unit application if no processing unit application exists.
| @mcp.prompt( | ||
| name="register_hardware", | ||
| description="Single task: provide the hardware topology — register a SCALEXIO/MicroAutoBox III/MicroLabBox II platform, import an .htfx file, or add a VEOS processing unit", | ||
| description="Single task: provide the hardware topology — register a SCALEXIO/MicroAutoBox III/MicroLabBox II platform, import an .htfx file, or add a processing unit application for VEOS", |
There was a problem hiding this comment.
Delete the "or add a processing unit application". You might add something like "Registering a hardware is not necessary if you want to create a Bus simulation container (BSC)"
| The runtime schema for `build_application` accepts `download`, `start`, and | ||
| `unload`. Use `download=false` when hardware download is not wanted. | ||
|
|
||
| The build generates one real-time application per processing unit application, |
There was a problem hiding this comment.
That is not correct. ConfigurationDesk creates one real-time application with 1..n processing unit applications. Each processing unit application runs on one processing unit. An application with 2..n processing unit application is a multi-processing-unit application that can run on a multi-processing-unit system.
Summary
Fix code and documentation related to Processing Unit Application handling:
add_application_processing_unittoadd_processing_unit_applicationand move it from hardware related module to application management related module.add_processing_unit_application.Validation
uv run ruff check .uv run ruff format --check .uv run pytest ConfigurationDeskMCP/testsuv run configurationdesk-mcp --list-tools