MNT: Name Conflict in `radius` property and input of `TankGeometry`
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:
-
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
radiuscould be changed toradius_function; - Allow for
**kwargsand allow the keyradiusso as to keep compatibility.
- Input parameter
-
Changing the property's name: an alternative is to change the property's name. I find this solution harder to execute without breaking changes.