Skip to content

refactor(simplex): extract results directly from dictionaries - #22

Merged
rasheedja merged 3 commits into
masterfrom
refactor/direct-result-extraction
Oct 4, 2026
Merged

rasheedja merged 3 commits into
masterfrom
refactor/direct-result-extraction

Conversation

@rasheedja

Copy link
Copy Markdown
Owner

Summary

  • Read basic-variable values directly from DictValue.constant when returning an optimum.
  • Remove the dictionary-to-tableau conversion that existed only to read the identical right-hand-side values.
  • Preserve slack/artificial-variable filtering and the minimization objective sign correction. Keep the public tableau conversion utility available.

This cleanup targets master independently; it does not require the correctness fixes to merge first. The map-fold and printer cleanups are already included in #18 and #19.

Validation

@rasheedja
rasheedja merged commit f7c7537 into master Oct 4, 2026
23 checks passed
@rasheedja
rasheedja deleted the refactor/direct-result-extraction branch October 4, 2026 10:10
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.

1 participant