Update example_opt_ipopt_4_ieacs4.jl

[IdarBarstad]

I recently downloaded julia and tried the example here after installing Flowfarm.jl. Several problems occurred (I've marked them with "IB" above in the code). My feeling is that this is because new, updated libraries are used. Perhaps some updates should be done?

[jaredthomas68]

@IdarBarstad Thanks for this pull request, and for pointing out the lines you've corrected so far. There are a few updates to the example that need to be made to match changes in the underlying FLOWFarm calls that need to get corrected as well as what you've already pointed out. I should be able to get this done this week and push up an updated Ipopt example.

In the meantime, my preferred way to use Ipopt with FLOWFarm is through SNOW.jl. You can take a look at the example script example_opt_snow.jl for how to do this.

[IdarBarstad]

Thanks for taking the time. I re-winded the Ipopt version to 0.7.0 and that sorted most of the issues. When running the optimization, however, an error was thrown saying something about getindex( ... -problem. I couldn't find where that came from. Still the same file I play with: example_opt_ipopt_4_ieacs4.jl

When using example linked directly from the webpage (using SNOW), I seem to get that to work OK. Cheers, Idar

On Tuesday, January 10, 2023 at 03:48:40 PM GMT+1, Jared Thomas ***@***.***> wrote:  

@IdarBarstad Thanks for this pull request, and for pointing out the lines you've corrected so far. There are a few updates to the example that need to be made to match changes in the underlying FLOWFarm calls that need to get corrected as well as what you've already pointed out. I should be able to get this done this week and push up an updated Ipopt example.

In the meantime, my preferred way to use Ipopt with FLOWFarm is through SNOW.jl. You can take a look at the example script example_opt_snow.jl for how to do this.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

[IdarBarstad]

Thanks for pointing out the example_opt_snow.jl file. I'm able to run this and a 10% improvement on AEP it says. Idar

On Tuesday, January 10, 2023 at 03:48:40 PM GMT+1, Jared Thomas ***@***.***> wrote:  

@IdarBarstad Thanks for this pull request, and for pointing out the lines you've corrected so far. There are a few updates to the example that need to be made to match changes in the underlying FLOWFarm calls that need to get corrected as well as what you've already pointed out. I should be able to get this done this week and push up an updated Ipopt example.

In the meantime, my preferred way to use Ipopt with FLOWFarm is through SNOW.jl. You can take a look at the example script example_opt_snow.jl for how to do this.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>