I.e. it would be better if the following throws an error:
Params(getPolymer(2),someRandomKwarg=1). Currently, it ignores the faulty kwarg.
This is because the kwargs are forwarded to both NumericalParams and OptionParams for convenience, which both ignore unknown kwargs.
The solution should ideally not break this behaviour
I.e. it would be better if the following throws an error:
Params(getPolymer(2),someRandomKwarg=1)
. Currently, it ignores the faulty kwarg. This is because the kwargs are forwarded to both NumericalParams and OptionParams for convenience, which both ignore unknown kwargs. The solution should ideally not break this behaviour