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

MNT: Name Conflict in radius property and input of TankGeometry #759

Open
phmbressan opened this issue Dec 20, 2024 · 1 comment
Open

MNT: Name Conflict in radius property and input of TankGeometry #759

phmbressan opened this issue Dec 20, 2024 · 1 comment
Labels

Comments

@phmbressan
Copy link
Collaborator

Description of the Issue

The TankGeometry class has the input parameter radius which is not saved on its __init__ as it is part of the standard of other rocketpy classes and assumed by StochasticModels.

To complicate matters, the class has a property with the name radius which is a Function of the input parameter of the same name (i.e. the input parameter per se is not saved, but its Function is a property).

Some Possible Solution Paths

The following approaches may be considered:

  1. Changing the input parameter: differentiating the property and the input parameter by changing the latter name is possible while keeping breaking changes at minimum, I believe:

    • Input parameter radius could be changed to radius_function;
    • Allow for **kwargs and allow the key radius so as to keep compatibility.
  2. Changing the property's name: an alternative is to change the property's name. I find this solution harder to execute without breaking changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Backlog
Development

No branches or pull requests

2 participants
@phmbressan and others