refactor(pumpkin-core): Extracting branchers as separate crate - #562

Open

ImkoMarijnissen wants to merge 10 commits into

mainfrom

refactor/extract-branchers

ImkoMarijnissen commented Sep 7, 2026

Contributor

This PR (using Claude) does the following:

Extracts the branchers from core
Uses State in the SelectionContext

ImkoMarijnissen added 2 commits

September 7, 2026 16:24


          refactor: original refactor of branchers out of core

1834f30


          refactor: use state in selection context

dbb6f83

ImkoMarijnissen commented

View reviewed changes

pumpkin-solver/src/lib.rs Outdated

ImkoMarijnissen added 2 commits

September 8, 2026 07:46


          refactor: remove Cow from SelectionContext

938c0eb


          refactor: fix test cases + change input to AlternatingBrancher

10aa51c

ImkoMarijnissen commented

View reviewed changes

pumpkin-crates/branchers/src/branchers/alternating/strategies/every_x_solutions.rs Outdated

ImkoMarijnissen commented

View reviewed changes

pumpkin-crates/branching/src/branching/alternating/strategies/until_solution.rs

ImkoMarijnissen commented

View reviewed changes

pumpkin-crates/branchers/src/value_selection/in_domain_interval.rs Outdated

ImkoMarijnissen commented

View reviewed changes

pumpkin-crates/branching/src/default_brancher.rs

ImkoMarijnissen commented

View reviewed changes

pumpkin-crates/core/src/api/mod.rs Outdated

ImkoMarijnissen commented

View reviewed changes

pumpkin-crates/core/src/branching/selection_context.rs Outdated

ImkoMarijnissen added 6 commits

September 8, 2026 08:36


          chore: do not use mapping when only a single variable is being created

294f699


          refactor: use solver-based creation methods for API

52fb59f


          chore: make get_solution_reference private again

755aaa5


          refactor: move TestRandom to branchers crate since it is only used there

4d9e9b4


          chore: remove unnecessary part of documentation

72aabfa


          refactor: rename from branchers to branching

44b10a4

ImkoMarijnissen requested a review from EmirDe

September 8, 2026 08:10

ImkoMarijnissen marked this pull request as ready for review

September 8, 2026 08:10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet