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

Update names of errinros_mod, chnrosnb_mod and fletcbv3_mod? #356

Closed
danphenderson opened this issue Dec 18, 2024 · 3 comments · Fixed by #357
Closed

Update names of errinros_mod, chnrosnb_mod and fletcbv3_mod? #356

danphenderson opened this issue Dec 18, 2024 · 3 comments · Fixed by #357

Comments

@danphenderson
Copy link
Contributor

The current names of these ADNLPModels are errinros, chnrosnb and fletcbv3; respectively.

In my use case, I am instantiating a set of ADNLPModels from a list of their names; but I run into issues with these models as their names do not correspond to the name of the file in the source code.

A fix would be to update these model's names to errinros_mod, chnrosnb_mod and fletcbv3_mod.

@danphenderson
Copy link
Contributor Author

Also, indef_mod.

@tmigot
Copy link
Member

tmigot commented Dec 18, 2024

Thanks @danphenderson for the issue. You are right, bwe should modify the problems' names.

For errinros_mod:

return ADNLPModels.ADNLPModel(f, x0, name = "errinros"; kwargs...)

and also the NLS variant:
return ADNLPModels.ADNLSModel!(F!, x0, 2 * (n - 1), name = "errinros-nls"; kwargs...)

This could also be tested, probably here:


@test nlp_ad.meta.name == String(pb)

strangely enough the name was correct in the dataframe describing the problems

:name => "errinros_mod",

@danphenderson Would you be interested in opening a PR to fix this?

@danphenderson
Copy link
Contributor Author

@tmigot, thanks for the additional instructions - PR submitted.

@tmigot tmigot linked a pull request Dec 20, 2024 that will close this issue
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

Successfully merging a pull request may close this issue.

2 participants