Feature/comet logging

[sherpan]

Description

Add comet as a potential logger for the repo

Changes

• Bug fix (non-breaking change which fixes an issue)
• Refactor (non-breaking change which refactors the code base)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

Checklist

• My code follows the pre-commit style and check guidelines of this project.
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing tests pass locally with my changes

[ashwinvaidya17]

@sherpan can you also have a look at https://github.com/openvinotoolkit/anomalib/runs/8065761026?check_suite_focus=true

[samet-akcay]

Thanks @sherpan ! I have few minor comments.

[sherpan]

Believe i resolved all of the comments!

[samet-akcay]

@sherpan, I was wondering if you could install pre-commit in your local environment, which would solve all these issues that you are having. Something like

pip install pre-commit
pre-commit run --all-files

This would fix the issues, after which you could push your changes.

[sherpan]

hey samet, yeah i think you are right! will do that next time? Think i am passing everything now?

[sherpan]

confused. here is what i am seeing on my side?

[samet-akcay]

That's peculiar. No worries, though. We'll sort out the rest. Thanks a lot for the contribution! Much appreciated!

[ashwinvaidya17]

Thanks for the changes

[ashwinvaidya17]

@sherpan There is probably something wrong with your environment. I am getting the same issues.

I have pushed the fix to this branch https://github.com/openvinotoolkit/anomalib/tree/fix/ashwin/linting. I cannot update your branch as I don't have permissions for your fork.

[ashwinvaidya17]

Or maybe we can merge this branch and I can open a new PR for https://github.com/openvinotoolkit/anomalib/tree/fix/ashwin/linting. @samet-akcay @djdameln ?