Fixes #167, `NormalizationParameter` defaults introduced in #161

oslocyclotronlab / ompy

A python implementation of the Oslo method

https://ompy.readthedocs.io

GNU General Public License v3.0

7 stars 7 forks source link

Fixes #167, `NormalizationParameter` defaults introduced in #161 #168

Closed fzeiser closed 3 years ago

fzeiser commented 3 years ago

optional arguments In #161 some additional parameters were added to NormalizationParameters. This caused trouble in NormalizerGSF, which assumed that all parameters/attributes in NormalizationParameters have to be set.
Sphinx docs: Workaround for documentation of non default members. Necessary due to https://github.com/astropy/sphinx-automodapi/issues/115

fzeiser commented 3 years ago

On the optional arguments: For now, I added a exclude_check_change attribute to mark the real optional arguments. Alternatively NormalizerGSF could be set to look just for some specific parameters it needs.