pytorch/ignite

Warning using ConfusionMatrix and setup_tb_logging

Chiusa

#829 aperta il 4 mar 2020

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Python (602 fork)batch import
enhancementhelp wantedmodule: metrics

Metriche repository

Star
 (4313 stelle)
Metriche merge PR
 (Merge medio 18g 23h) (19 PR mergiate in 30 g)

Descrizione

❓ Questions/Help/Support

logger configured by setup_tb_logging warns when a ConfusionMatrix metric is attached to the evaluator

tensorboard_logger.py:129: UserWarning: TensorboardLogger output_handler can not log metrics value type <class 'torch.Tensor'>
  "TensorboardLogger output_handler can not log " "metrics value type {}".format(type(value))

The reason is setup_tb_logging (via setup_any_logging) defines OutputHandler using metric_names="all" https://github.com/pytorch/ignite/blob/a22a0f5e909ac70d2a1f76a60b6e84b2134f196c/ignite/contrib/engines/common.py#L248-L254

What do you think to expose metric_names in the API to be able to fix the warning ?

Guida contributor

Direzione di ricerca
Examine the setup tb logging function in ignite/contrib/engines/common.py (lines 248-254). The OutputHandler is instantiated with metric names="all", causing a warning when ConfusionMatrix returns tensors. Consider adding a parameter to setup tb logging to allow users to specify metric names, thereby avoiding the warning. Ensure backward compatibility and check if similar changes are needed in setup any logging.
Tech stack
python
Dominio
machine learning
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonPyTorchIgnite basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70