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

added setting, pre consume to filter object if the config exists #473

Closed
wants to merge 2 commits into from

Conversation

AlxEnashi
Copy link
Contributor

Summary of the changes

added an optional setting (maybe should be default 0?) to filter too short lightcurves.
havent addressed the border case when no alert is left in the input.

Observations

<-- Add some notes to keep in mind !-->

If the change is BREAKING, please give more details about the breaking changes

<-- What change breaks what and how !-->

Components that need updates and steps to follow

<-- Link repos or other PRs !-->

About this PR:

  • You added the necessary documentation for the team to understand your work.
  • You linked this PR to the corresponding issue or user story.
  • Your changes fulfill the Definition of Done of the issue or user story.
  • Your changes were tested with manual testing before being submitted.
  • Your changes were tested with automatic unit tests.
  • Your changes were tested with automatic integration tests.

Screensshots

<-- Add some screenshots if necessary !-->

@AlxEnashi AlxEnashi linked an issue Aug 13, 2024 that may be closed by this pull request
@AlxEnashi AlxEnashi closed this Aug 19, 2024
@AlxEnashi AlxEnashi deleted the feat/filter_short_lc_classificer branch August 19, 2024 13:47
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.

Add filter short light curves lc_classification
1 participant