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

Update autoencoder.py #10

Open
wants to merge 17 commits into
base: morpheus-22.08
Choose a base branch
from

Conversation

gbatmaz
Copy link
Collaborator

@gbatmaz gbatmaz commented Sep 12, 2022

adding early stop

add early stop
mdemoret-nv and others added 16 commits September 13, 2022 11:50
* Reducing the number of cpu -> gpu calls

* Adding comparisons

* MOre debugging

* Fixed final bug

* More fixes discovered during testing

* Cleaning up the repo. Removing debugging comparisons
* add option to preset categories

* set the index of the prediction df to match the input df
Raises a `ValueError` when `x` isn't one of the unexpected types.

Avoids an `UnboundLocalError: local variable 'result' referenced before assignment` error

Not sure if this is specific to our fork or if it should be contributed upstream.

Authors:
  - David Gardner (https://github.com/dagardner-nv)

Approvers:
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: #4
Addressing: nv-morpheus/Morpheus#498

Batching the evaluation of the input dataframe in the `get_anomaly_score_losses` function helps with the high GPU memory usage issue.

Authors:
  - https://github.com/hsin-c

Approvers:
  - David Gardner (https://github.com/dagardner-nv)
  - Eli Fajardo (https://github.com/efajardo-nv)
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: #5
…ses (#6)

1 of the 2 PRs for the issue: nv-morpheus/Morpheus#497
This PR checks the `DFP to optionally use modified zscore (MAD)` checkbox.

Authors:
  - https://github.com/hsin-c

Approvers:
  - https://github.com/gbatmaz
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: #6
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.

5 participants