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

Minor refactor #587

Merged
merged 3 commits into from
Sep 26, 2022
Merged

Minor refactor #587

merged 3 commits into from
Sep 26, 2022

Conversation

ashwinvaidya17
Copy link
Collaborator

Description

  • Minor refactor

djdameln and others added 2 commits September 26, 2022 13:42
* fix patchcore image-level score computation

* docstring and comment

* remove default value for n_neighbors

* torch.Tensor -> Tensor
Copy link
Contributor

@samet-akcay samet-akcay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Would it be possible to make this change to #576 ? Otherwise, we'll need to update it too?

@ashwinvaidya17 ashwinvaidya17 changed the base branch from main to documentation-refactor September 26, 2022 14:07
@ashwinvaidya17
Copy link
Collaborator Author

@samet-akcay I've changed the target but the documentation-refactor branch is missing one comment from main. I'll address the merge conflict meanwhile.

@samet-akcay samet-akcay merged commit 6b5f262 into documentation-refactor Sep 26, 2022
@samet-akcay samet-akcay deleted the ashwin/minor_fixes_hpo branch September 26, 2022 18:34
samet-akcay added a commit that referenced this pull request Sep 27, 2022
* Add benchmark to tutorial

* Move export to tutorials

* Move hpo to tutorials

* Move inference to tutorials

* Move logging to tutorials

* Create installation in tutorials

* Create training to tutorials

* Create tutorials index

* Update conf.py file

* Add anomalib logos to logos directory

* Add data docs

* Add algos

* Add model docs

* Add reference api

* Remove blank line in metrics

* Add reference guide

* Add how to guides

* Add developer guide

* Add blog to how-to-guide

* Remove guides directory

* Add train custom data to how-to-guides

* Fix typos

* Add notebooks to how-to-guides

* Add anomalib favicon

* Add missing algo descriptions

* Rename Reference to Reference Guide

* Add how to add a new model

* fix typos

* Merge PR 544

* Minor refactor (#587)

* 🛠 Fix PatchCore image-level score computation (#580)

* fix patchcore image-level score computation

* docstring and comment

* remove default value for n_neighbors

* torch.Tensor -> Tensor

* Minor refactor

Co-authored-by: Dick Ameln <dick.ameln@intel.com>
Co-authored-by: Ashwin Vaidya <ashwinitinvaidya@gmail.com>

* Address Dicks comments

Co-authored-by: Ashwin Vaidya <ashwin.vaidya@intel.com>
Co-authored-by: Dick Ameln <dick.ameln@intel.com>
Co-authored-by: Ashwin Vaidya <ashwinitinvaidya@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants