Skip to content

LOBSTER Report - no schema#524

Open
ThomasDebobenVispiron wants to merge 36 commits intomainfrom
vtt/lobster_report_v2
Open

LOBSTER Report - no schema#524
ThomasDebobenVispiron wants to merge 36 commits intomainfrom
vtt/lobster_report_v2

Conversation

@ThomasDebobenVispiron
Copy link
Contributor

Remove schema from .lobster files with version 5.
Only write Item's to get remove Requirement, Implementation and Acticity in the future.
For transition period the old versions are still supported.

Introduce new yaml config file for LOBSTER report.

For future use we want to support a yaml file as config file fore the lobster_report tool.
During implementation and testing phase the known .conf file still can be used.
For future use we want to support a yaml file as config file fore the lobster_report tool.
During implementation and testing phase the known .conf file still can be used.
For future use we want to support a yaml file as config file fore the lobster_report tool.
During implementation and testing phase the known .conf file still can be used.
For future use we want to support a more flexible configuration.
Therefor we remove the schema (requrement, implementation, code) from the lobster files. Version without schema is 5.
All items are now from base Item class.

Remove schema from .lobster files with version 5.
Only write Item's to get remove Requirement, Implementation and Acticity in the future.
For transition period the old versions are still supported.
Remove schema from .lobster files with version 5.
Only write Item's to get remove Requirement, Implementation and Acticity in the future.
For transition period the old versions are still supported.
@ThomasDebobenVispiron ThomasDebobenVispiron requested a review from a team as a code owner January 13, 2026 15:08
Remove schema from .lobster files with version 5.
Only write Item's to get remove Requirement, Implementation and Acticity in the future.
For transition period the old versions are still supported.
fix pylint
fix bazel integration
Copy link
Contributor

@mugdhadhole1 mugdhadhole1 left a comment

Choose a reason for hiding this comment

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

Since both old and new code currently run and the old one will be removed later, it will be very helpful if a comment exists marking where the old code exists.

Copy link
Contributor

@mugdhadhole1 mugdhadhole1 left a comment

Choose a reason for hiding this comment

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

Please update the README to reflect the latest changes in the tools. The example section also needs to be revised to match the updated implementation.

@mugdhadhole1
Copy link
Contributor

I'm attaching an example where I attempted to generate the HTML report.
The report generated, but the content was empty.
example.zip

@hoe-jo hoe-jo self-requested a review February 26, 2026 10:51
Copy link
Contributor

@hoe-jo hoe-jo left a comment

Choose a reason for hiding this comment

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

This PR is way to huge to be reviewed, please split it in content and tooling changes.

Review will be done from team Sarthi

adjust system tests and lobster_html_report to write pure items without kind
adjust system tests and lobster_html_report to write pure items without kind
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