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

Prebuilt libraries disk usage #39

Open
gsauthof opened this issue May 1, 2022 · 3 comments
Open

Prebuilt libraries disk usage #39

gsauthof opened this issue May 1, 2022 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@gsauthof
Copy link
Contributor

gsauthof commented May 1, 2022

When cloning the repository over a slow connection I noticed that it contains a bunch of prebuilt libraries:

$ du -chs NMSIS/Library/
117M	NMSIS/Library/
117M	total

I'm kind of afraid that those prebuilt libraries will bloat this repository in the future even more, when each change to them basically adds another 100 MBs or so to the git index.

Is it really necessary to include those artifacts?

Perhaps there are alternatives such as using Git Large File Storage or building them from source like the rest?

@fanghuaqi fanghuaqi self-assigned this May 5, 2022
@fanghuaqi fanghuaqi added the enhancement New feature or request label May 5, 2022
@fanghuaqi
Copy link
Member

Hi @gsauthof , currently this prebuilt libraries are used by demo_dsp example, and with this prebuilt NMSIS DSP/NN libraries at specified version of NMSIS, user no need to build this library.

If I add this as Git LFS objects, it might introduce more complicated usage for some basic user which is not familiar with git, and maybe in future I can add it as git lfs, and provide a better solution when I create regular releases, and normal user should use the regular release version of SDK, and advance user directly use with git version.

Thanks
Huaqi

@fanghuaqi
Copy link
Member

fanghuaqi commented Nov 7, 2023

With the release 0.5.0, more variant of prebuilt libraries are included, so the size of library now is 425M.

$ du -d1 -h NMSIS/Library/
361M    NMSIS/Library/DSP
64M     NMSIS/Library/NN
425M    NMSIS/Library/

@fanghuaqi
Copy link
Member

In future release of Nuclei SDK, we will directly include NMSIS DSP/NN source code, instead of library, so user can directly try with the source code, not prebuilt library with big size, and can build with different arch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants