Skip to content
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

fwi_2D_small_overthrust_lbfgs.jl software papers example #255

Open
scott314159 opened this issue May 31, 2024 · 1 comment
Open

fwi_2D_small_overthrust_lbfgs.jl software papers example #255

scott314159 opened this issue May 31, 2024 · 1 comment

Comments

@scott314159
Copy link

I tried to run fwi_2D_small_overthrust_lbfgs.jl and got the following error:
#############################################################
ERROR: LoadError: MethodError: no method matching optimize(::Opt, ::PhysicalParameter{Float32, 2})

Closest candidates are:
optimize(::Opt, ::AbstractVector{<:Real})
@ NLopt ~/.julia/packages/NLopt/w0c7n/src/NLopt.jl:634
###############################################

I the issue seems to come from line 66:
(minf, minx, ret) = optimize(opt, copy(model0.m))

I found a different example which has optimize syntax:
(minf, minx, ret) = optimize(opt, vec(model0.m.data))

Which seems to work but I am not sure if this is the appropriate fix.

@mloubout
Copy link
Member

vec(model0.m.data)). is fine yes, looks like some type mismatch that is now popping up with the PhysicalParameter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants