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

Upgrade component output conditional generation #91

Open
mkorbel1 opened this issue Sep 16, 2024 · 0 comments
Open

Upgrade component output conditional generation #91

mkorbel1 opened this issue Sep 16, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@mkorbel1
Copy link
Contributor

Motivation

There are a variety of ways to have optional output ports, for example:

  • A bool to control whether it gets generated with a nullable getter
  • An optional argument to the constructor that connects to an output (requires width matching)
  • A getter with a side-effect of lazily creating the port (and associated logic) when it's requested

These all have pros and cons in both implementation and use cases

Desired solution

Debate these, figure out if there's a consistent solution that's good for all cases vs. multiple depending on where it is

Alternatives considered

No response

Additional details

No response

@mkorbel1 mkorbel1 added the enhancement New feature or request label Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant