-
-
Notifications
You must be signed in to change notification settings - Fork 688
a bijectionist's toolkit #35060
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
a bijectionist's toolkit #35060
Changes from 1 commit
Commits
Show all changes
64 commits
Select commit
Hold shift + click to select a range
6d7ab9c
initial commit
mantepse 6149eb2
make the linter happier
mantepse 9d4bfb9
remove useless assignment
mantepse 6ee34e1
make initialisation more efficient
mantepse 1a14bcf
consistently name variable
mantepse 0901ec7
slightly improve documentation
mantepse c39d653
non-copying intersection, to save memory when there are almost no res…
mantepse 2f0bb0e
Merge branch 'develop' of trac.sagemath.org:sage into t/33238/a_bijec…
mantepse 321ba43
start to cache solutions
mantepse 306395c
finish implementation of cache
mantepse 1a8e564
Merge branch 'develop' of trac.sagemath.org:sage into t/33238/a_bijec…
mantepse 1ac0978
add some documentation and doctests, slightly simplify code
mantepse 47945ac
add missing documentation in table of contents
mantepse 19c3d8f
mark doctests as long, slightly simplify logic
mantepse eca857e
slightly simplify, more doctests
mantepse a04d146
doctest _find_counter_example
mantepse 3508426
doctest add_distribution_constraints and add_intertwing_relation_cons…
mantepse 0ac618c
doctest _preprocess_intertwining_relations, _solution, _show_bmilp, _…
mantepse d50b62c
expand docstring of main class
mantepse 288e391
copy milp instead of adding and removing constraints
mantepse d57c8e5
derandomize a test, mark example as random
mantepse 7353fb5
correct typo, remove useless assignment
mantepse db850f0
add and remove constraints instead of copying the whole program
mantepse ec3271c
fix problem with SCIP, add timings
mantepse dfdc6ba
use convert for MixedIntegerLinearProgram.get_values
mantepse d0836cb
include information on MILP also in public documentation
mantepse 563af98
remove unnecessary copy
mantepse 590bae2
add possibility to constrain to involutions
mantepse da0655b
correct indentation
mantepse 03b1fb8
better handling of empty constraints
mantepse a3364de
move _show to the _BijectionistMILP class
mantepse 9d83a6c
remove unused code for backends that do not support removing constraints
mantepse 864029d
make _solution a public method of _BijectionistMILP, simplify solve
mantepse 64101a8
eliminate _initialize_new_bmilp
mantepse 2d34911
slightly simplify logic of _forced_constant_blocks, use defaultdict
mantepse c2f0062
slight simplification
mantepse 78968ae
copy (instead of deepcopy) should be correct
mantepse 6ddaeae
slightly simplify _preprocess_intertwining_relations
mantepse 60286e6
slightly improve non_copying_intersection
mantepse c38f5b9
add some internal documentation
mantepse 6fb06f0
untangle _preprocess_intertwining_relations
mantepse 706056e
add possibility to require a homomesy
mantepse d8a0663
Merge branch 'u/mantepse/mixedintegerlinearprogram_add_constraint__re…
mantepse 4e90280
Merge branch 'u/mantepse/allow_to_remove_no_constraints' of trac.sage…
mantepse 65b0c4a
preserve the cache of solutions after computing the optimal constant …
mantepse 21cb54f
make _BijectionistMILP.solution the only entrypoint
mantepse a573bb4
remove unnecessary calls to list in doctests
mantepse 6868261
move iterator over all solutions to _BijectionistMILP
mantepse 61e97bc
merge _solve, solution and __iter__
mantepse db8947b
pycodestyle stuff
mantepse 9678717
Merge branch 'develop' of trac.sagemath.org:sage into t/33238/a_bijec…
mantepse 53506aa
rename pseudo_inverse to quadratic
mantepse d61c630
Merge branch 'u/mantepse/a_bijectionist_s_toolkit' of https://github.…
mantepse fefe54a
Merge branch 'develop' of github.com:sagemath/sage into develop
mantepse 31bfb69
fix docstrings, simplify some tests
mantepse 24e4084
change lambda to def
mantepse e7e063d
Merge branch 'develop' into develop
mantepse dc3009a
reviewer's suggestions
mantepse a457b99
Merge branch 'develop' of github.com:mantepse/sage into develop
mantepse 6a738b6
Merge branch 'develop' into develop
mantepse cb2c82a
Merge branch 'develop' into develop
mantepse 109f9e4
Merge branch 'develop' into develop
mantepse f87c437
sort result of doctest to avoid random failures
mantepse a72bd65
Merge pull request #1 from mantepse/bijectionist
mantepse File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
pycodestyle stuff
- Loading branch information
commit db8947bfe5f116492b6fc864b36e6d9054bc34cb
There are no files selected for viewing
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think calling
keys()here is unnecessary, but not 100% sure.