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 detection is running #295

Merged
merged 132 commits into from
Feb 20, 2024
Merged

Conversation

JonasFrey96
Copy link
Collaborator

Tested anomaly detection:

  • To use anomaly detection one has to simply change the model within the cfg
  • I added an additional parameter within the confidence generator, I will explain to you in person the following drawing:

image

Issues:

  • within the default.yaml we have a parameter that is used to overwrite the confidence generator used for anomaly_detection and the normal one in our confidence - this is fine but may also be confusing given that you may want to parameterize those two things separately.
  • check with robin given that the learning performance is sub-ideal for the flow model, which is independent of the confidence_generator / scaling.
  • for the confidence_generator, we are currently running the latest_measurment, but I would assume using the last 5 measurements or some window may be better. I think this should be implemented, but I did not test it.

@JonasFrey96 JonasFrey96 merged commit bdab047 into main Feb 20, 2024
3 of 4 checks passed
@JonasFrey96 JonasFrey96 deleted the dev/anomaly_detection_is_running branch February 20, 2024 18:38
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