Draft
Conversation
sshin23
reviewed
May 21, 2024
| @@ -0,0 +1,230 @@ | |||
|
|
|||
| include("shapes/circle.jl") | |||
| for b1 in 2:dom.ELEMS+1; | ||
| lcon=-Inf, | ||
| ucon=0.0, | ||
| ) |
Member
There was a problem hiding this comment.
These constraints can benefit significantly from constraint! if n is large.
Consider reformulating this as
c = ExaModels.constraint(
core,
- z[1] for b1 in 2:dom.ELEMS+1;
lcon=-Inf,
ucon=0.0,
)
ExaModels.constraint!(
core,
b1 -1 => integral[b1+1, e1] for b1 in 2:dom.ELEMS+1
)
* solved error in pde_models * solved constraint error in pde_models * fixed summation problem in pde_models * added semicolonn to scopf * remove local test in pde_models.jl * add catmix draft * added chain,methanol,minsurft and fixed catmix and pde_models * added polygon, tetra, torison and triangle * reduce variables integral in pde_models * changed channel.jl from JuMP to ExaModels * finished glider.jl * merged conflicts in pde_models * solved constraint error in pde_models * solved error in pde_models * added chain,methanol,minsurf,catmix,polygon,tetra,torison,triangle and fixed pde_models * reduce variables integral in pde_models * changed channel.jl from JuMP to ExaModels * finished glider.jl * merged conflicts in pde_models * solved constraint error in pde_models * add rectangular coordinates in opf.jl * removing division in constraints * removing division in constraints * removing division in constraints * solved merge issue * fixed bounds for voltage * fixed bounds in opf.jl * Update bounds for voltage in opf.jl The original constraint was not correct because it should be the square of magnitude. * included more problems for testing * add parameters into model to avoid testing errors * solving input error in tetra_model * fixed input error in pde_models * fixed input for runtests.jl * fixed name of the problem torsion.jl * Delete src/torison.jl Delete wrong name model * Changing function names in ExaModelsExamples.jl Torsion has replaced torsion * added test cases to runtest.jl * added LuksanVleckModels in the package * changed incorrect package name and make it more accessible * modified formulation in tetra.jl
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds the
dirichlet,henonandlane_emdeninstances from the COPS benchmark. These PDE-constrained instances are the most challenging ones, and pause difficulties to the automatic differentiation.Unfortunately, the compilation time is currently too long in ExaModels, and I was not able to solve these three instances despite trying different workarounds.
My observations with other AD backends are: