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

Repo size #3

Open
dzid26 opened this issue Jun 10, 2023 · 6 comments
Open

Repo size #3

dzid26 opened this issue Jun 10, 2023 · 6 comments

Comments

@dzid26
Copy link

dzid26 commented Jun 10, 2023

The repo size is approaching 1GB.
Can you move the /releases (analyzer.exe, etc) to github releases and rewrite git history while still not many people has forked it?

I also like putting PCB gerbers, pdfs and ibom in release as well.

@dzid26
Copy link
Author

dzid26 commented Jun 10, 2023

I did it in my fork.

java -jar bfg.jar --delete-files stepper_monitor-2023-06-10_153440.zip
java -jar bfg.jar --delete-files stepper_monitor-2023-06-10_145755.zip
java -jar bfg.jar --delete-files nrf_sniffer_for_bluetooth_le_4.1.1.zip
java -jar bfg.jar --delete-files ibom.html
java -jar bfg.jar --delete-files positions.csv
java -jar bfg.jar --delete-files gerber.zip
java -jar bfg.jar --delete-files bom.csv
java -jar bfg.jar --delete-files bootloader.bin
java -jar bfg.jar --delete-files firmware.bin
java -jar bfg.jar --delete-files partitions.bin
java -jar bfg.jar --delete-files analyzer
java -jar bfg.jar --delete-files analyzer.exe
java -jar bfg.jar --delete-files bluetooth_spec_Core_v4.2.pdf
java -jar bfg.jar --delete-files bluetooth_spec_Core_v5.3.pdf
git push --force
git reflog expire --expire=now --all && git gc --prune=now --aggressive

Now it's 157MB + releases.

@zapta
Copy link
Owner

zapta commented Jun 10, 2023

Hi @dzid26,

Can you reply here with your your suggestions for repo structure changes and I will check with @AdamV3D if it will not disturb his fork at https://github.com/AdamV3D/ble_stepper_motor_analyser. A text description should do, e.g. 'move file x to location z'.

BTW, the esptool is included in the release to allow users to perform firmware upgrade in the field.

Also, currently the releases covers four aspects of this project, hardware (Kicad), firmware that runs on the devices, python software that runs on the PC, software tools for upgrade such as esptool.

Thanks,
Zapta

@zapta
Copy link
Owner

zapta commented Jun 10, 2023 via email

@dzid26
Copy link
Author

dzid26 commented Jun 10, 2023

I moved esptool to the tools folder, because it made sense.

I think releases should be fairly rare, so it's ok if they contain Kicad, firmware, and PC - all together. If someone wants the latest stuff they will probably be compiling things themselves anyway.

Yes, Github Actions should be able to generate firmware and even gerberas. I have examples of such generation (with resulted files at the bottom) however I never figured out how to make it appear in releases. Still useful if someone just needs the latest bin.

@AdamV3D 's fork would break in the sense that automated merging would stop being possible as it will have "unrelated history".
I actually merged their changes already in my fork, but I removed the branding.

@AdamV3D
Copy link

AdamV3D commented Jun 10, 2023

I'm not too fussed about what goes where, as long as there is some logic to it and you let me know what I need to change.

@dzid26
Copy link
Author

dzid26 commented Jun 10, 2023

@zapta
If you are happy with the file list that I pruned from the repo, you can simply add my fork to your remotes git remote add dzid_repo https://github.com/dzid26/ble_stepper_motor_analyzer.git, then git reset --hard be5677df7288d70ab088b6f44482682492ed3467 (this will override any local modifications) to be on my be5677d (or this one 237ca53), and at the end you git push --force to Github.

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