Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Struct support - vspec2ddsidl #238

Open
erikbosch opened this issue Feb 16, 2023 · 3 comments
Open

Struct support - vspec2ddsidl #238

erikbosch opened this issue Feb 16, 2023 · 3 comments

Comments

@erikbosch
Copy link
Collaborator

This issues concerns struct support for vspec2ddsidl.

Required work:

  • Define/propose how it shall be handled (possibly in this PR)
  • Implement and test
  • Make sure that expected output format is described in documentation
  • If feasible, add at least one test case checking output

VSS Struct support is expected to be officially released for VSS 4.0, late April. To identify and manage remaining a work number of issues have been created to facilitate more detailed discussion and to identify volunteers for implementing/reviewing/testing.

@SebastianSchildt will check if the ones originally contributing vspec2ddsidl are interested

@happikin
Copy link

I am willing to take up this feature addition. Is there any document or diagram which may help me briefly understand the flow of code in the py files.

@erikbosch
Copy link
Collaborator Author

I am willing to take up this feature addition. Is there any document or diagram which may help me briefly understand the flow of code in the py files.

Great!

We do not have that much documentation, there is a little at:

To get know a little about the general structure of exporters/tools you can look at the test case at https://github.com/COVESA/vss-tools/blob/master/tests/generators/example_generator.py

I did however notice that we already did some work on DDSIDL support in #262, so this issue is not really up to date. Please check if the current level support fits your needs. I noticed that not all test cases have been updated to include DDSIDL struct support, I will do some edits and mention you so that you can get an overview of the current level of support.

@erikbosch
Copy link
Collaborator Author

See #340 for an example on how DDSIDL struct support works as of today, that could help identifying improvement areas. If we think it is good enough we should close this issue

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

No branches or pull requests

2 participants