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.

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

IRKGL16-SIMD #71

Merged
merged 1 commit into from
Nov 9, 2024
Merged

IRKGL16-SIMD #71

merged 1 commit into from
Nov 9, 2024

Conversation

ChrisRackauckas
Copy link
Member

This is a rebased and squashed version of #69 . The issue is the ipynb checkpoints were bloating the repo, causing unremovable memory. That is fixed in the squashed commits that should mostly prevent against repo bloat. It would probably be good to not include all of the notebooks here but if they aren't changed often.

@mikelehu

IRKGL16-SIMD

IRKGL16-SIMD

IRKGL16-SIMD

IRKGL16-SIMD

IRKGL16-SIMD

IRKGL16-SIMD

remove extra data

remove more checkpoints

more checkpoint clean up
@ChrisRackauckas
Copy link
Member Author

Unless you squash before merging, old binaries are never removed from a Git repo. The major factor at play here are the notebooks and the images inside of them. Each time they are changed, the old versions are still there in the history. With this squash I removed all of the checkpoints which keeps it a bit tame:

Screenshot 2024-11-09 at 9 45 08 AM

so ~100mb. But we should probably watch that growth. If they aren't changing much or often from this point then I think that's okay (and you cannot remove that without changing old git hashes which effects versioning). Though for the future benchmarks, we may want to make this in SciMLBenchmarks in order to avoid the repo growth (this is why the benchmarks were moved out of DifferentialEquations.jl, the repo is over 1gb and we cannot remove the history 😅)

@ChrisRackauckas ChrisRackauckas merged commit 4e0dac2 into master Nov 9, 2024
4 of 5 checks passed
@ChrisRackauckas ChrisRackauckas mentioned this pull request Nov 9, 2024
5 tasks
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