Pr 1421#1439
Pr 1421#1439prasadtalasila wants to merge 11 commits intoINTO-CPS-Association:feature/distributed-demofrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## feature/distributed-demo #1439 +/- ##
============================================================
+ Coverage 87.80% 96.45% +8.65%
============================================================
Files 26 126 +100
Lines 287 3641 +3354
Branches 18 598 +580
============================================================
+ Hits 252 3512 +3260
- Misses 35 126 +91
- Partials 0 3 +3 see 130 files with indirect coverage changes
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This pull request updates documentation across the DTaaS platform, primarily focusing on formatting improvements, new documentation pages, and updated external links. The changes enhance the user documentation with detailed guides for website settings, DevOps features, and concurrent execution capabilities.
Key Changes:
- Added comprehensive documentation for website settings configuration and DevOps features
- Fixed table formatting inconsistencies across multiple documentation files
- Updated external resource links to reflect newer versions of presentations and videos
Reviewed changes
Copilot reviewed 39 out of 102 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| servers/execution/runner/README.md | Fixed markdown table formatting with proper spacing |
| mkdocs.yml, mkdocs-github.yml | Restructured navigation to add new documentation sections for settings, capabilities, and concurrent execution |
| docs/user/website/settings.md | New comprehensive guide for DTaaS configuration parameters |
| docs/user/website/index.md | Added configuration check section with placeholder URL |
| docs/user/website/images/ | Added new supporting images for settings documentation |
| docs/user/servers/execution/runner/README.md | Added Linux platform warning and fixed table formatting and JSON syntax |
| docs/user/motivation.md | Updated links to newer presentation and video resources (version 20250605) |
| docs/user/library/devops/ui.md | New documentation for reusable assets and DevOps integration |
| docs/user/gitlab/tutorial.md | Added warning about default branch requirements and link to settings |
| docs/user/examples/*.md | Standardized table formatting across all example documentation |
| docs/user/digital-twins/devops/ui.md | Major restructuring with new Create, Manage, and Execute tab documentation |
| docs/user/digital-twins/devops/execution-settings.md | New detailed guide for allowed configuration values with troubleshooting |
| docs/user/digital-twins/devops/concurrent-execution.md | New guide for running multiple digital twins simultaneously |
| docs/thanks.md | Fixed table formatting |
| docs/redirect-page.html | Added version 0.7.0 to documentation versions |
| docs/index.md | Updated main page links to newer resources |
| docs/developer/*.md | Updated table formatting and npm package publication information |
|
|
||
| If you have trouble logging into the application, please check for | ||
| the correct configuration of the website by going to | ||
| <https://foo.com/config/user>. The following display must appear |
There was a problem hiding this comment.
The URL "https://foo.com/config/user" in the documentation appears to be a placeholder. This should be updated to either a real example URL or a more descriptive placeholder like "https://your-dtaas-instance.com/config/user" to make it clear that users need to substitute their own domain.
docs/user/digital-twins/devops/ui.md
Outdated
|
|
||
| The left-side menu shows the possibility of creating | ||
| [necessary structure](../../digital-twins/devops/file-structure.md) | ||
| and elements for a new DT. Each reusable asset selected for this new DT' |
There was a problem hiding this comment.
There's a typo in the sentence. It should be "Each reusable asset selected for this new DT" instead of "Each reusable asset selected for this new DT'". The trailing apostrophe should be removed.
|
| Uses markdown-aware parsing to avoid replacing content in code blocks. | ||
| """ | ||
|
|
||
| import os |
There was a problem hiding this comment.
Import of 'os' is not used.
Improvements over PR #1421 and the following updates: