Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Replace coo_sym_prod! by coo_sym_add_mul! #182

Closed
amontoison opened this issue Jun 20, 2024 · 0 comments · Fixed by #183
Closed

Replace coo_sym_prod! by coo_sym_add_mul! #182

amontoison opened this issue Jun 20, 2024 · 0 comments · Fixed by #183
Labels
enhancement New feature or request

Comments

@amontoison
Copy link
Member

amontoison commented Jun 20, 2024 •
edited
Loading

For a MathOptNLPModel, we use coo_sym_prod! and we should replace it by coo_sym_add_mul!.

coo_sym_prod! is in NLPModels.jl:
https://github.com/JuliaSmoothOptimizers/NLPModels.jl/blob/bfd4ceafec9ad6f58cbf11a4a88f2e61767ce701/src/nlp/utils.jl#L92-L116
@amontoison amontoison added the bug Something isn't working label Jun 20, 2024
@amontoison amontoison mentioned this issue Jun 20, 2024
Merged
@amontoison amontoison added enhancement New feature or request and removed bug Something isn't working labels Jun 20, 2024
@amontoison amontoison changed the title grad! and hprod! are not well tested Replace coo_sym_prod! by coo_sym_add_mul! Jun 20, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace coo_sym_prod! by coo_sym_add_mul! amontoison/NLPModelsJuMP.jl
1 participant
@amontoison