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

Improve readability of sai_api_gen.py for SAI header file generation. #470

Merged
merged 15 commits into from
Dec 8, 2023

Conversation

r12f
Copy link
Collaborator

@r12f r12f commented Dec 7, 2023

This PR will not change any generated SAI header files but only update the sai_api_gen.py file for better readability.

After this change, all header files are lib files are output with the exactly the same format:

SAI folder:
image

Lib:
image

@KrisNey-MSFT
Copy link
Collaborator

Reorganized sections to more easily see the hierarchy and become more obvious. Much easier changes in the future; for this one no functionality will change, all produced files will remain unchanged.

@KrisNey-MSFT KrisNey-MSFT merged commit 5bdd504 into sonic-net:main Dec 8, 2023
4 checks passed
r12f added a commit to r12f/DASH that referenced this pull request Dec 8, 2023
…sonic-net#470)

* temp checkin.

* temp checkin.

* Update SAI to reference.

* table data parser.

* DASH API set object.

* minor update.

* Table keys and types.

* table action and param.

* fix v4_or_v6 fields.

* Simplify template based file generation code.

* Revert to current SAI

* SAI Generator.

* Fix header generation issues.

* Fix lib generation issue.

* Fix ignored tables.
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.

3 participants