Mikkel Friis-Møller (9ed9a4c6) at 18 Mar 10:56
Mikkel Friis-Møller (9ed9a4c6) at 18 Mar 10:56
corrected the way opt_settings are passed to ScipyOptimizeDriver
Mikkel Friis-Møller (9ed9a4c6) at 18 Mar 10:12
corrected the way opt_settings are passed to ScipyOptimizeDriver
Mikkel Friis-Møller (74bfa388) at 15 Feb 09:57
Mikkel Friis-Møller (949c532c) at 15 Feb 09:48
fixed DistanceConstraintAggregation with different types of boundar...
... and 5 more commits
I was running into an issue when using the DistanceConstraintAggregation class, for the purpose of optimizing with SGD as explained in this tutorial https://topfarm.pages.windenergy.dtu.dk/TopFarm2/notebooks/sgd_slsqp_comparison.html.
I was able to reproduce exactly the same error by modifying the tutorial. The tutorial has a rectangular boundary, which is instantiated as
constraint_comp = XYBoundaryConstraint(boundary, 'rectangle')
it also works if the type is 'convex_hull', but if constraint_comp is any type that "might be" non-convex like 'polygon' or 'multi_polygon', it fails with the following error:
RuntimeError: Collected errors for problem 'problem4': : The source and target shapes do not match or are ambiguous for the connection 'pre_constraints.xy_bound_comp.boundaryDistances' to 'constraint_aggregation_comp.boundaryDistances'. The source shape is (9,) but the target shape is (9, 4)
I wonder if it is expected that this should not work for non-convex boundaries, or if it is a bug that can be solved. If it is expected behavior, then I think that a more explicit error message should be presented to the user.
I attach a full notebook (which is just a cut and paste of the tutorial with the aforementioned line modified)
Thanks in advance
S
Mikkel Friis-Møller (53844f60) at 11 Dec 13:36
Mikkel Friis-Møller (ba6ef954) at 11 Dec 13:36
Neighbouring turbines
Mikkel Friis-Møller (53844f60) at 11 Dec 12:29
add notebook header
Mikkel Friis-Møller (d1b67745) at 11 Dec 11:43
pep8
Mikkel Friis-Møller (89403b76) at 11 Dec 10:49
pep8
Mikkel Friis-Møller (283cd32c) at 08 Dec 13:44
added option to include neighbouring turbines
... and 2 more commits
Maria Virginia Sarcos Araujo (11d85893) at 04 Dec 13:57
Mikkel Friis-Møller (5a2f69db) at 04 Dec 13:56
Update constraint_aggregation.py