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

Adding reaction network name as option? #762

Open
TorkelE opened this issue Jan 14, 2024 · 0 comments
Open

Adding reaction network name as option? #762

TorkelE opened this issue Jan 14, 2024 · 0 comments

Comments

@TorkelE
Copy link
Member

TorkelE commented Jan 14, 2024

Currently, network names are added as

@reaction_network rn_name begin
    ...
end

Given that we now support options, it would be possible to do

@reaction_network begin
    @name rn_name
    ...
end

It doesn't really matter, both approaches work. Generally, the first one is neater. However, it is a bit funny that reaction network names is the one input type which is inputted differently compared to all other ones. Moving to the second one would allow us to have a single uniform approach.

I have no strong opinions, but at least something one can consider.

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

1 participant