Skip to content

solver: reduce items in edge optimizations#51503

Merged
alalazo merged 1 commit intospack:developfrom
alalazo:solver/improve-optimization-statements
Nov 4, 2025
Merged

solver: reduce items in edge optimizations#51503
alalazo merged 1 commit intospack:developfrom
alalazo:solver/improve-optimization-statements

Conversation

@alalazo
Copy link
Copy Markdown
Member

@alalazo alalazo commented Nov 3, 2025

Edge optimizations are a tie-breaker only for nodes that are allowed to have duplicates. As such, we can reduce the number of items entering the minimizations only to the ones that might be duplicated.

radiuss.optimization.csv
radiuss.develop.csv

comparison

@alalazo alalazo marked this pull request as ready for review November 3, 2025 20:28
tgamblin
tgamblin previously approved these changes Nov 3, 2025
@tgamblin
Copy link
Copy Markdown
Member

tgamblin commented Nov 3, 2025

@alalazo can you rebase this?

Edge optimizations are a tie-breaker only for nodes that are
allowed to have duplicates. As such, we can reduce the number
of items entering the minimizations only to the ones that
might be duplicated.

Signed-off-by: Massimiliano Culpo <[email protected]>
@alalazo alalazo force-pushed the solver/improve-optimization-statements branch 2 times, most recently from c624c2f to ce77e70 Compare November 4, 2025 06:32
@alalazo alalazo enabled auto-merge (squash) November 4, 2025 08:06
@alalazo alalazo merged commit 7581ea2 into spack:develop Nov 4, 2025
56 of 61 checks passed
@alalazo alalazo deleted the solver/improve-optimization-statements branch November 4, 2025 08:10
hippo91 pushed a commit to hippo91/spack that referenced this pull request Nov 4, 2025
Edge optimizations are a tie-breaker only for nodes that are
allowed to have duplicates. As such, we can reduce the number
of items entering the minimizations only to the ones that
might be duplicated.

Signed-off-by: Massimiliano Culpo <[email protected]>
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 this pull request may close these issues.

3 participants