Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Memory complexity of the temporal problem #101

Closed
Marius1311 opened this issue Feb 21, 2022 · 4 comments
Closed

Memory complexity of the temporal problem #101

Marius1311 opened this issue Feb 21, 2022 · 4 comments
Assignees
@MUCDK
Labels
benchmark Benchmarking results

Comments

@Marius1311
Copy link
Collaborator

Similar to #91, this looks at the memory complexity of moscot vs. WOT with online = True/False flavours. Let's start with CPU memory if that's easier - we'll keep GPU memory for later then.
@Marius1311 Marius1311 added the benchmark Benchmarking results label Feb 21, 2022
@Marius1311 Marius1311 mentioned this issue Feb 21, 2022
Closed
@Marius1311
Copy link
Collaborator Author

Now that ott-jax/ott#10 is merged we should see the linear memory complexity for online mode. Of course, low-rank should also show linear memory complexity.

@michalk8
Copy link
Collaborator

michalk8 commented Feb 22, 2022 •
edited
Loading

Sharing what I've send Zoe/Dominik: below is a figure for offline using a 32k x 16x coupling

profile

This would explain why it's still the same as online, will verify.

@Marius1311
Copy link
Collaborator Author

Hi @michalk8, could you please open a new issue in the OTT/Jax repo (since ott-jax/ott#9 was closed) to tell Marco and Laetitia about our current problems, and what we know already? Maybe they have some hints for us. Thanks!

@Marius1311
Copy link
Collaborator Author

I suppose this is solved.

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

@MUCDK MUCDK

Labels
benchmark Benchmarking results
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Marius1311 @michalk8 @MUCDK