Refactor argument handling: simplify logic by using arg_len variable #915
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Provide an overview of the implemented solution or the fix and elaborate on the modifications.
Refactored the argument handling logic in the simulation code. Introduced a variable
arg_len
to simplify the checks on the length ofargs
and reduce redundancy. The new structure first checks the length ofargs
and branches accordingly to call the appropriate simulation methods (_simulate_power_law
,_simulate_model
,_simulate_model_string
,_simulate_power_spectrum
, or_simulate_impulse_response
) based on the type and number of arguments passed.Is there a new dependency introduced by your contribution? If so, please specify.
No new dependencies have been introduced.
Any other comments?
This refactor improves the readability and maintainability of the code.