Mashiro

Results 68 comments of Mashiro

TextLoggerHook Handle the `by_epoch` attribute in https://github.com/open-mmlab/mmcv/blob/75ae2009545b086c8304d9d16150565ecbdc8565/mmcv/runner/hooks/logger/text.py#L163 Therefore you can control the iteration-based or epcoh-based logging format. Although this will work for the rest of the logger hooks as well,...

Hi~ Have you followed the steps in https://mmcv.readthedocs.io/en/latest/deployment/tensorrt_plugin.html?highlight=trt? It seems that you've not TensorRT correctly

Hi, It seems you are using tensorRT 8.10. How about trying TensorRT-7.2.1.6 and following the steps in docs. ![image](https://user-images.githubusercontent.com/57566630/179195022-62160d1e-e955-481e-892f-9b9424c6c1f8.png)

LGTM. But it seems that the propagate attribute of Logger causes the duplicate log. Can set `logger.propagate` to False solve the problem of duplicate logs? This way we don't need...

The `init_process_group` function in Pytorch 1.8 adds a `StreamHandler` to the root logger, so if `get_logger` is called after `init_process_group`, the log level of the root logger will be set...

LGTM. I‘ll do further testing in the downstream codebase.

@sleep3r thanks for your reply~. If we set default StreamHandler for `logging.root` in `get_logger`, It seems there will be no message shown in the terminal by calling logging.warning('xxx'): ```python import...

> @HAOCHENYE I can't reproduce the behaviour with warning logging. Setting `propagate=False` leads to fallen tests, and I don't have much about this question too. > @HAOCHENYE I can't reproduce...

Thanks for your contribution! It seems that the previous implementation has some bugs. But `self.meta = checkpoint['meta']` still seems weird, the current `exp_name` and `env_info` will be overwritten, is this...