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

merge master into branch for AL loop #354

Merged
merged 8 commits into from
Oct 25, 2023

Conversation

emilleishida
Copy link
Contributor

IMPORTANT: Please create an issue first before opening a Pull Request.
Linked to issue(s): Closes 344

If this is a new release, did you issue the corresponding schema in fink-client?

otherwise you'll break the livestream service!

What changes were proposed in this pull request?

update branch for new AL model

How is the issue this PR is referenced against solved with this PR?

How was this patch tested?

emilleishida and others added 8 commits October 11, 2023 06:16
* add trained model for first AL loop with SSO

* add model trained in 40 samples

* new model for AL loop
* Apply the condition to remove unwanted objects with null features

* Bump to 4.10
* add trained model for first AL loop with SSO

* add model trained in 40 samples

* new model for AL loop

* Update requirements.txt

update actsnfink

* Update run_test.yml

update actsnfink

* Update processor.py

change test to allow 3 points on the rise, not 4

* Update requirements.txt

update actsnfink

* Update run_test.yml

update actsnfink for tests

* Update processor.py

revert tests to match actsnfink
* Add customisation to feature extractor in SN Ia. Update the test suite for AL loop.

* Update constructor

* Fix test suite
* Joblib models updated. Pkl file converted to joblib

* AGN classifier now reads joblib

* Fixed path

* Use joblib instead of pickle to load the model
@emilleishida emilleishida merged commit 86a63f2 into 333-al-loop-in-real-time-with-sso Oct 25, 2023
4 checks passed
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.

3 participants