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

remove contrib from handlers imports #1

Closed

Conversation

leej3
Copy link
Owner

@leej3 leej3 commented Mar 22, 2024

Follow on from pytorch#3204. Separated for better clarity in the file diffing.

* migrate ignite.contrib.handlers

move loggers and other class from contrib to ignite.handlers
make changes in a backwards compatible manner
fix circular imports that happened during the migration in ignite/__init__.py
and ignite/handlers/__init__.py

* update tests

move tests to ignite.handlers along with fixtures and changing the
imports

add a test to ensure deprecation warnings for ignite.contrib.handlers

* improve docs

mark as deprecated and provide link to migrated functionality

* make alternative fix for circular import

* add modules to contrib handlers namespace

previously this occured as a side effect, adding it explicitly now.
leej3 and others added 3 commits March 22, 2024 14:30
* add Entropy metric

* fix error in torch.randint

* update Entropy metric to support other shapes

* Update ignite/metrics/entropy.py

Co-authored-by: vfdev <vfdev.5@gmail.com>

* update test of Entropy metric

* Update ignite/metrics/entropy.py

* format code

* fix error in converting Tensor to ndarray

---------

Co-authored-by: vfdev <vfdev.5@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants