-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add max, min and std operators to multimodel #602
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cheers Javi @jvegasbsc - I was supposed to do this but it completely slipped my mind 🍺 It's worth pointing out and I think we need to add it to the documentation that these are local (min, max)
and not global
If we add a comment about it, we should do it for every operator. All are local |
yeah I think we should say that, lemme add it to the doc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! @mattiarighi Could you please test?
I have some installation issues at the moment (see #535) and cannot test. |
@mattiarighi Did you manage to solve your installation issues? |
Yes, sorry for being absent lately, I'm very busy with other projects. |
Seems to work as expected, thanks for adding this feature! |
Added several operators to the multimodel, as requested by @mwjury
Tasks
yamllint
to check that your YAML files do not contain mistakesIf you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.
Closes #563