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

anomaly pseudo step #481

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

anomaly pseudo step #481

wants to merge 8 commits into from

Conversation

Bastianciito
Copy link
Contributor

@Bastianciito Bastianciito commented Aug 29, 2024

Summary of the changes

<-- Please add a brief summary of the changes !-->

The pseudo step added, "lc_anomaly_step," is a copy of "lc_classification_step" but includes some functions to insert "classifications" (scores) into a custom database when the model is anomaly detector type. Database credentials are environment variables and have the following names:

ANOMALY_PASSWORD
ANOMALY_DB_NAME
ANOMALY_HOST
ANOMALY_PORT
ANOMALY_SCHEMA
ANOMALY_USER

Observations

There is no observations

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

The following test were ignored :

test -> integration -> test_step_anomaly

this step was renamed to -> skip_step_anomaly

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 !-->

No SS

@Bastianciito Bastianciito requested a review from AlxEnashi August 29, 2024 18:49
@Bastianciito Bastianciito self-assigned this Aug 29, 2024
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.

2 participants