Skip to content

Add a possibility to plugin user gets wall properties#398

Merged
alexandrebbruno merged 21 commits intomasterfrom
fb-ASIM-5688-exporting-variables-APB-plugin
Aug 28, 2024
Merged

Add a possibility to plugin user gets wall properties#398
alexandrebbruno merged 21 commits intomasterfrom
fb-ASIM-5688-exporting-variables-APB-plugin

Conversation

@alexandrebbruno
Copy link
Copy Markdown
Contributor

PR description:

  • Adds get_wall_properties, the method that a plugin developer user will use to get wall properties.

Comment thread src/alfasim_sdk/alfasim_sdk_api/api.h
Comment thread src/alfasim_sdk/alfasim_sdk_api/api.h Outdated
Copy link
Copy Markdown
Contributor

@ggrbill ggrbill left a comment

Choose a reason for hiding this comment

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

Nice job!

Please, check the appearance of the new documentation.

Comment thread src/alfasim_sdk/alfasim_sdk_api/api.h Outdated
Comment thread src/alfasim_sdk/alfasim_sdk_api/api.h Outdated
Comment thread src/alfasim_sdk/alfasim_sdk_api/api.h Outdated
Comment thread src/alfasim_sdk/alfasim_sdk_api/api.h Outdated
Copy link
Copy Markdown
Contributor

@prusse-martin prusse-martin left a comment

Choose a reason for hiding this comment

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

Just some documentation updates.

Comment thread src/alfasim_sdk/alfasim_sdk_api/api.h Outdated
Comment thread src/alfasim_sdk/alfasim_sdk_api/api.h
Comment thread src/alfasim_sdk/alfasim_sdk_api/api.h Outdated
Comment thread src/alfasim_sdk/alfasim_sdk_api/detail/api_pointers.h
@alexandrebbruno alexandrebbruno force-pushed the fb-ASIM-5688-exporting-variables-APB-plugin branch from 674395e to 893528f Compare August 27, 2024 11:18
@sonarqubecloud
Copy link
Copy Markdown

@alexandrebbruno alexandrebbruno merged commit 0220420 into master Aug 28, 2024
@alexandrebbruno alexandrebbruno deleted the fb-ASIM-5688-exporting-variables-APB-plugin branch August 28, 2024 13:47
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.

6 participants