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

[models] do not include models with the code #299

Open
JulienPeloton opened this issue May 10, 2023 · 0 comments
Open

[models] do not include models with the code #299

JulienPeloton opened this issue May 10, 2023 · 0 comments
Labels

Comments

@JulienPeloton
Copy link
Member

JulienPeloton commented May 10, 2023

The models are getting too big! We can keep them inside the repo, but they should not be distributed via pypi (limit of 100MB already reached).

Strategy 1

  • each processor should have an argument to get the path
  • models should be copied to this location

This has the advantage to keep pypi to distribute the code, but there is an extra burden to synchronize with the models...

Strategy # 2

Alternatively, we can stop using pypi for this repo, and install using the URL directly (using releases):

pip install git+https://github.com/astrolabsoftware/fink-science@RELEASE

Of course, we will hit some limit later (about 5GB). I personally prefer this one though...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Dependency
Development

No branches or pull requests

1 participant