Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update deep_svdd.py #607

Closed
wants to merge 5 commits into from
Closed

Update deep_svdd.py #607

wants to merge 5 commits into from

Conversation

modanesh
Copy link

Resolving loss.backward() error. It resolves issue #606

All Submissions Basics:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully ran tests with your changes locally?
  • Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?
  • Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

  • Have you created a .py in ~/pyod/models/?
  • Have you created a _example.py in ~/examples/?
  • Have you created a test_.py in ~/pyod/test/?
  • Have you lint your code locally prior to submission?

@yzhao062
Copy link
Owner

yzhao062 commented Dec 1, 2024

thanks for the PR -- also by any chance could you redirect it to the development branch? thanks, Yue

@modanesh modanesh changed the base branch from master to development December 2, 2024 18:03
@modanesh
Copy link
Author

modanesh commented Dec 2, 2024

@yzhao062 thanks for the review. I just redirected my PR to the development branch.

@modanesh modanesh closed this Dec 6, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants