Conversation
no longer relevant with the change of hypervolume calculations
|
API looks quite clean! Can you add a prefix or f-string or something for the name? T-strings might even be useful here. Old way also still works like, if you need more flexibility on parameters, right? |
|
T-strings might be useful in the future, but they are Python 3.14. I need to work with 3.12 for HPC reasons. yes old way will still work. |
|
I'm a bit confused looking at the implementation. The description shows: def my_func(decisions:np.array[float]):
....
model.levers = [RealParameter(decisions, shape=(3,)),]Several questions:
|
|
Its wip, so I was doing stuff locally. The latest commit should provide some additional clarity, but there is still more work to be done. 1: The new API contained a mistake. I had forgotten the bounds. |
many tests are still failing and new tests are still needed
|
Thanks for explaining. Might be useful to mark the PR as "Draft", to reflect its state. |
This PR adds a new
shapekeyword argument toParameter, closing #303.Before this PR, each uncertainty/lever needs to match with a keyword argument. However, if you have many identical parameters such as in case of the intertemporal lake problem, this is annoying to specify both at the model level and at the workbench level. This PR adds a shape argument to
Parameterto resolve this issue. This PR also ensures that this shape parameter is properly handled in exploration, optimization, and when storing results.Old API
New API