-
Notifications
You must be signed in to change notification settings - Fork 9
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
(feat): Linear GENOT #662
Merged
Merged
(feat): Linear GENOT #662
Changes from all commits
Commits
Show all changes
334 commits
Select commit
Hold shift + click to select a range
f86eb0e
fix jaxsampler
MUCDK ac3c781
fix jaxsampler
MUCDK 385bcfd
fix jaxsampler
MUCDK cfebc4d
fix tests
MUCDK e056042
add plot_convergence
MUCDK a7af86b
remove jit from _compute_unbalanced marginals
MUCDK c8cbe9d
fix sinkhorn_divergence
MUCDK ff145d1
adapt tox.ini file
MUCDK 9e241db
shape mismatch fixed without precommit
AlejandroTL 3cc279f
remove print statement
MUCDK c10c5b6
finish merge
MUCDK 74adb1e
adapt callbacks and rename tag `cost` to `cost_matrix` (#426)
MUCDK a282376
Feature/correlation test (#423)
MUCDK 7360819
fix sankey return statement (#428)
MUCDK e08f63f
Bump version: 0.1.0 → 0.1.1
MUCDK 728537c
fix return statements
MUCDK f2153e9
add save tests
MUCDK 4f1d26e
fix return type in mpl (#432)
MUCDK 0ffc730
Simplify linear operator (#431)
michalk8 cee2466
Explicitly jit the solvers (#433)
michalk8 6b11188
Feature/interpolate colors sankey (#434)
MUCDK e28816f
Remove `FGWSolver` (#437)
michalk8 3ff63be
fix bug in SinkhornProblem (#442)
MUCDK 865ad5a
fix pre commits
MUCDK 2571935
make push/pull always use source/target (#443)
MUCDK 3d22677
fix strip plotting in sankey (#445)
MUCDK b5370e0
Feature/spearman correlation (#444)
MUCDK 34ef904
Delete logo.png
MUCDK 9806854
Feature/plot order (#453)
MUCDK 50dbdca
Expose marginal kwargs for `moscot.temporal` and check for numeric ty…
MUCDK 46e1c65
adapt plot_convergence (#454)
MUCDK 61f5481
Bug/docs generic analysis mixin (#455)
MUCDK 082c879
Docs/improvements (#456)
MUCDK a53ef22
remove uns_key from set_plotting_vars (#458)
MUCDK 3a9f992
resolve `fig referenced before assignment` (#460)
MUCDK c323cb5
move generic mixins tests to problems` (#461)
MUCDK edb3c11
Tests/spatiotemporalproblem (#464)
MUCDK e3c3911
Feature/move taggedarray (#457)
MUCDK fdc5d54
add marginal_kwargs to prepare method of TemporalNeuralProblem
MUCDK 859700c
fix to scaling in
lucaeyring 22a48f5
Revert "fix to scaling in"
lucaeyring dfa2d82
fix to scaling argument in marginal_kwargs
lucaeyring 4bcd966
updated conditional not pipeline
lucaeyring cb7bc2d
merge into condot branch
lucaeyring ddb1772
merge into condot branch
lucaeyring 9636f1c
incoporated comments
lucaeyring f27dab0
incoporated comments
lucaeyring 816b084
incoporated comments
lucaeyring 1eb60f4
removed new_adata for push/pull
lucaeyring 2b6686f
Merge pull request #9 from theislab/condot_revamp
MUCDK 9e053ca
Merge pull request #7 from theislab/conditional_not_precommit
lucaeyring 0d96dbe
[ci skip] start docs
MUCDK 83e6d71
added temporal neural test
lucaeyring df346d9
[ci skip] continue docs
MUCDK ec0ab31
continue docs
MUCDK fbd4181
continue docs
MUCDK 10799ec
change validation epsilon
MUCDK 7f94ef9
fixed error when not computing wasserstein baseline
lucaeyring 1421c65
fixed error when not computing wasserstein baseline
lucaeyring b828752
Merge pull request #11 from theislab/feature/docs
lucaeyring 1b584ae
Merge branch 'main' into temporal_neural_test
lucaeyring d37c7d3
Merge pull request #12 from theislab/temporal_neural_test
lucaeyring dd106af
correct typo
MUCDK 453922c
fix bug
MUCDK 298f8fb
added neural tests
lucaeyring 491a9a4
[ci skip] draft CondNeuralOutput
MUCDK e6b1f9a
include CondDualPotentials and CondDualSolver
MUCDK 77fea48
merge moscot main restructuring
lucaeyring b1316bc
fixes to main merge
lucaeyring 1ff844f
fix typo
MUCDK 29ff674
fix test_cell_transition_subset_pipeline
MUCDK 518a8e5
fix tests
MUCDK 2a5bcd7
update conditionalDualPotentials
MUCDK 6594d43
update conditionalDualPotentials
MUCDK 2c43b07
fix most pre-commit hooks and fix tests
MUCDK 594e2f9
fix pandas version to <2.0
MUCDK ebc7877
fix tests for non-conditional solvers
MUCDK df0b6a8
merge continue_docs
MUCDK db7b2c0
continue
MUCDK 3220f9e
fix
MUCDK 4fb6e51
continue fixing
MUCDK fdf2882
fix ICNN setup
MUCDK 35db509
fix tests
MUCDK 65850ae
Merge pull request #18 from theislab/neural_tests_local
MUCDK b7d8a00
swap role of f and g, such that push/pull is correct again
MUCDK ea7187e
[ci skip] restructure to include more general neural solvers
MUCDK 0c9a587
[ci skip] restructure ICNNs to allow passing instances of ICNN
MUCDK d3a529d
adapt tests
MUCDK aba225d
Filled in Monge Gap structure
gocato d5005b1
Added Monge Gap paper to documentation
gocato 2e0fb4a
Ammend PointCloud Import
gocato f328056
Merge remote-tracking branch 'origin/feature/add_monge_gap' into feat…
gocato 4692dc5
Update _utils.py
gocato 36b1953
Merge remote-tracking branch 'origin/feature/add_monge_gap' into feat…
gocato 66e5794
Solve compatibility issue with ProblemKind
gocato 2ad39a8
Solve missing Import
gocato 16a797d
Fix call to deprecated function
gocato 7d63082
Fix style and comment issues
gocato 9aff229
add callback, swap f & g
lucaeyring 8f91e4a
add callback, swap f & g
lucaeyring 2d63db7
add callback, swap f & g
lucaeyring 5452485
Merge pull request #26 from theislab/feature/add_callback
MUCDK 6f413a2
Merge pull request #27 from theislab/dev
MUCDK 43b873d
intermediate save
MUCDK 602b8a7
intermediate save
MUCDK b85c8e0
intermediate save
MUCDK de6facc
intermediate save
MUCDK a32038c
partially resolve precommit errors
MUCDK fa9d6cf
[ci skip] fix merge conflicts
MUCDK 501b018
resolve conflict
MUCDK 0419670
remove pairwise policy
MUCDK ce7667a
add neural dependencies
MUCDK f97b08c
add neural dependencies
MUCDK 243c73a
add flax
MUCDK 4516612
fix _call_kwargs
MUCDK 68e7bf3
fix marginal kwargs
MUCDK 536f681
remove monge gap solver
MUCDK 3185371
clean condneuralsolver
MUCDK 582ad43
[ci skip] introduce new data container for joint neural problems
MUCDK 7e3d4f7
add conditions in distirbutioncontainer
MUCDK 51915db
resolve unfreeze/freeze
MUCDK 47508b3
enable pretraining and weight clipping
MUCDK 5363d98
make dicts compatible with older python versions
MUCDK 0dd6b8b
resolve precommit errors partially
MUCDK 21f3309
resolve precommit errors partially
MUCDK 3bbb4da
adapt tests
MUCDK 2cced99
[ci skip] draft unbalancedNeuralMixin
MUCDK aa49c10
[ci skip] fix naming of posterior marginals
MUCDK 16d3204
[ci skip] add MLP_marginals
MUCDK c61633c
adapt neural output to incorporate learnt rescaling functions
MUCDK 2dc6ff3
fix _solve in neuraldualsolver
MUCDK badd57b
incorporate feedback
MUCDK bd983fd
fix distributioncollection class
MUCDK b00e2b7
unify _split_data
MUCDK 79f050e
fix tests
MUCDK 5c19f1f
fix some precommit hooks
MUCDK 4e964e5
make neural dependencies optional
MUCDK 517db5f
make neural dependencies optional
MUCDK 742f02c
delete old files
MUCDK d763de4
adapt pyproject.toml
MUCDK 2c64e3b
adapt pyproject.toml
MUCDK 73c7830
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 6d40537
[ci skip] adjust _format_params
MUCDK 9483e69
adapt neuraldualsolver to be more similar to ott-jax
MUCDK 32300e5
adapt neuraldualsolver
MUCDK 88b9a59
TODO: make JaxSampler return conditions
MUCDK dd74215
add basic neural test
MUCDK dce2b21
[ci skip] intermediate save
MUCDK d2ece68
adapt neuraldualsolver and finish tests for neural backend
MUCDK 3bd674f
[ci skip] TODO: re-iterate on initialisation of neural solver
MUCDK 61e3a01
adapt distributioncontainer
MUCDK b949d6f
fix dict bug
MUCDK e90934c
resolve passing of arguments in solver call methods
MUCDK 295eb6e
[ci skip] adapt `solve` in `CondOTProblem`
MUCDK 08fcec2
adapt tests and valid loader conditions
MUCDK 868146a
adapt neural backend tests
MUCDK f84bd31
fix mypy errors
MUCDK e74e0a4
Merge branch 'main' into feature/moscot_not
MUCDK 9482ccc
make basesolveroutput to basediscretesolveroutput
MUCDK 182bab7
move `to` to BaseSolverOutput`
MUCDK 166bd49
adapt transport_matrix docs
MUCDK fea3ac9
adapt transport_matrix docs
MUCDK 7669f7b
adapt tests
MUCDK e53f9c0
adapt tests
MUCDK cc6bfb5
update unbalancedness mixin
MUCDK a438629
use implementation from moscot
MUCDK 3f422bf
uncomment unused code
MUCDK 3e9dfc5
before passing states to loss-fn
MUCDK f4b7c76
intermediate save
MUCDK 4152baf
adapt neuraldualsolver
MUCDK 06e55c0
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 0b46bae
resolve some / not all pre commit errors
MUCDK 4566eb9
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] c981af7
(wip): tests run, code swapped out for now
ilan-gold 8763321
(wip): `NeuralSolver`s implemented minus quad/linear
ilan-gold c2312e3
(wip): begin more generic problem
ilan-gold 7f09740
(wip): more refactoring to pass arguments to GENOT
ilan-gold 10c1102
(chore): remove more kantorovich
ilan-gold 4a7a79e
(chore): update branch to moscot neural + first test moving to solving
ilan-gold f10be61
(fix): split data remains in numpy
ilan-gold 03bec6b
(fix): push/pull api
ilan-gold a99231a
(fix): make push test work
ilan-gold b1f3ea4
(feat): allow for custom optimziers
ilan-gold e3b31d6
(chore): remove unclear test
ilan-gold 06376a1
(refactor): change to composition API
ilan-gold 0f955d5
(refactor): start towards model-specific problems
ilan-gold f1a0718
(chore): clean up all unnecessary classes
ilan-gold 94441f0
(chore): updating to moscot latest
ilan-gold 2583c56
Merge branch 'main' into ig/neural_solvers
ilan-gold 591f3b4
(chore): remove (hopefully) final ICNN vestiges
ilan-gold 96bd942
Merge branch 'main' into ig/neural_solvers
ilan-gold 37c3757
(chore): more cleanup
ilan-gold 5b4afdd
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 85ac491
Merge branch 'main' into ig/neural_solvers
ilan-gold 25844d7
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] a0f94a4
(fix): pass pre-commit hooks
ilan-gold f017c9e
Merge branch 'ig/neural_solvers' of github.com:theislab/moscot into i…
ilan-gold 6e85255
(chore): remove duplicatec docs
ilan-gold 133ed51
(chore): add torch for testing
ilan-gold 556f6ce
(fix): add ott jax branch as dep
ilan-gold 4bea0d2
(fix): repo name
ilan-gold e0a5628
(chore): remove unbalanced, update api, fix tests + drive by typing fix
ilan-gold d09a9eb
(feat): first pass at neural mixin
ilan-gold 6ac3d3f
(chore): add my name to todos
ilan-gold b2faf8a
(fix): conditions left out if not necessary
ilan-gold 79e76e1
(feat): logs and fix conditional attr
ilan-gold fbfd93f
(fix): add `seed` to call_kwargs so reproducibility works
ilan-gold ff27786
(chore): remove `is_conditional` business
ilan-gold 7e90a53
(fix): create hidden dims arg for velocity field
ilan-gold 1dbd78c
(chore): raise not implemented error for `pull`
ilan-gold f64d4ff
(fix): default args
ilan-gold 050360c
(fix): add explicit policy
ilan-gold 556a815
(fix): allow iteration to continue
ilan-gold d18154a
(chore): add star policy to GENOT
ilan-gold 24ee61e
(chore): notebooks
ilan-gold 77cd780
Merge branch 'main' into ig/neural_solvers
ilan-gold 83f9fb1
(chore): remove deps
ilan-gold a4ec4f9
(chore): remove unnecessary spaces
ilan-gold 0145b5b
(chore): simplify quad handling
ilan-gold 98177c1
(fix): need to require `optax`/`flax`
ilan-gold ea3dc93
(fix): use `ott-jax[neural]`
ilan-gold b9b66d5
(chore): fix docs
ilan-gold 16e45a1
(fix): small test fixes
ilan-gold 4cbc912
(chore): small notebook changes
ilan-gold cb04bef
(fix): broken link in citation
ilan-gold 884996d
(chore): make notebook dependent on ci
ilan-gold 0f353fc
(fix): small todos just to push something
ilan-gold 341068c
(fix): variable is a string
ilan-gold 64f89ea
(fix): pass environment variable to tox
ilan-gold 85342f4
(fix): actually pass through
ilan-gold 0098193
(fix): hidden dims ci
ilan-gold 6f9890a
(fix): re-add notebook
ilan-gold 7860056
Merge branch 'main' into ig/neural_solvers
ilan-gold 46e5fc2
(chore): make`recall_target` and `aggregate_to_topk`
ilan-gold bcd288c
(chore): fix default arguments
ilan-gold 822628d
(chore): `project_transport_matrix` -> `project_to_transport_matrix`
ilan-gold 5835390
(fix): remove dead `NeuralAnalysisMixin` code
ilan-gold d245cb0
Merge branch 'ig/neural_solvers' of github.com:theislab/moscot into i…
ilan-gold 90e9fc6
Merge branch 'main' into ig/neural_solvers
ilan-gold e9cd90e
(feat): allow custom `data_match_fn`
ilan-gold cf11513
(fix): inherit from `MutableMapping` instead of `dict`
ilan-gold 0f91c4e
Merge branch 'main' into ig/neural_solvers
ilan-gold ab51550
(Fix): docs
ilan-gold 2ce5c0b
(fix): notebooks
ilan-gold 9b38b65
(fix): docs reference
ilan-gold 60e074c
(fix): remove `attr`
ilan-gold c8a9cbd
(fix): erroneous change
ilan-gold b97a2b4
(fix): remove empty
ilan-gold d3b31ec
(fix): notebooks again?
ilan-gold 2678705
(chore): ok?
ilan-gold 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
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
Oops, something went wrong.
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 am not sure what these mean. Is this intentional or was it left from debugging
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 these are warnings from the RTD that should be ignored base on the static class rendering file. But they are indeed too many I'm not sure if it's wanted.
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.
Not sure, what to do here. We can't import from
MutableMapping
forDistributionCollection
without re-implementing all the abstract methods, which seems like a waste. The other option would be to remove theDistributionCollection
class since it is used only forrepr
/cleanness of API. @MUCDK this code was pulled from your starting point, so perhaps you can chime in.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'd be fine with keeping it as it is if it's not too much of an issue. @selmanozleyen wdyt?
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.
If typing.Dict doesn't work as well I guess using those ignores is a better solution than re-implementation.