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

Git history is getting huge due to Jupyter Notebooks #129

Open
timtreis opened this issue Aug 7, 2024 · 4 comments
Open

Git history is getting huge due to Jupyter Notebooks #129

timtreis opened this issue Aug 7, 2024 · 4 comments

Comments

@timtreis
Copy link
Member

timtreis commented Aug 7, 2024

Just cloned the repo and noticed that it was taking quite some time, downloading 550 MB for a fairly low-LOC repo:
image

I wonder if there's any merit towards switching to a fork/squash-merge approach instead of the branch based? Realistically, that history is only ever going to become larger.

Counterpoint:

  • This repo is not really something many people outside devs clone so it might be overengineering.

wdyt @giovp @LucaMarconato ?

@LucaMarconato
Copy link
Member

I wonder if what is proposed here is an acceptable solution: https://stackoverflow.com/questions/1209999/how-to-use-git-to-get-just-the-latest-revision-of-a-project.

@timtreis
Copy link
Member Author

timtreis commented Aug 7, 2024

I don't think that having these extra 500 MB on disk is the issue (mildly inconvenient at best), but realistically this history will very quickly grow even larger in size if we keep working on it without squashing these Jupyter Notebook commits - same for spatialdata-notebooks.

Maybe we could implement a "squash-and-merge-only" rule to counter-act that a little bit?

@LucaMarconato
Copy link
Member

That could be a good approach @giovp wdyt?

@giovp
Copy link
Member

giovp commented Aug 21, 2024

It would be interesting to decrease the size of this repo by squashing commits, not sure how to do it though. The solution you pointed to @LucaMarconato is interesting for dev perspective indeed. And also @timtreis you are right that the whole purpose of having the notebooks repo separate is that we knew this would happen in the future.

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

No branches or pull requests

3 participants