Willcox-Research-Group / rom-operator-inference-Python3

Operator Inference for data-driven, non-intrusive model reduction of dynamical systems.
https://willcox-research-group.github.io/rom-operator-inference-Python3
MIT License
67 stars 30 forks source link

Set default regularizer to float value #57

Closed Algopaul closed 5 months ago

Algopaul commented 6 months ago

Simply sets the default regularizer to 0.0 instead of 0. My linter complained as shown in the screenshot attached; re-setting the default removed this message.

Screenshot 2024-05-10 at 10 07 23
shanemcq18 commented 5 months ago

Thanks @Algopaul, this should merge in easily. Your linter may differ from what this project uses (are you using mypy?), but upcoming changes will introduce tox to make the linting process a little more standard.

Algopaul commented 5 months ago

Great! Thank you. I am using pyright; thanks for letting me know!