Zhong-Yu Li comments

Results 14 comments of


                                            Zhong-Yu Li

Checkpoint with Instance Branch

Hello, are there any plans to release those models?

Support receptive field search of CNN models. (TPAMI paper rf-next)

Hello, we have fixed the issues following your suggestions. Thanks. @zhouzaida

Support receptive field search of CNN models. (TPAMI paper rf-next)

> Could we add some unit tests to cover those modifications? It is nice to add some unit tests. But I wonder who should do this job, me or you?...

Support receptive field search of CNN models. (TPAMI paper rf-next)

Ok, I'll add the corresponding unit tests within a few days.

Support receptive field search of CNN models. (TPAMI paper rf-next)

Above issues have been fixed.

Support receptive field search of CNN models. (TPAMI paper rf-next)

Are there any other issues we should do apart from unit tests? @zhouzaida

Support receptive field search of CNN models. (TPAMI paper rf-next)

Hello, we have added unit tests for the rfsearch. Please let me know if there are any issues or anything things I can do. Thanks. @zhouzaida

Support receptive field search of CNN models. (TPAMI paper rf-next)

All above issues are fixed.

Support receptive field search of CNN models. (TPAMI paper rf-next)

> Please move the file `tests/test_cnn/test_rfsearch_operator.py` to `tests/test_cnn/test_rfsearch/test_operator.py` and those modifications in `tests/test_runner/test_hooks.py` to `tests/test_cnn/test_rfsearch/test_search.py` I have completed it.

Support receptive field search of CNN models. (TPAMI paper rf-next)

> Hi @lzyhha , have you tested this latest commit with [open-mmlab/mmdetection#8191](https://github.com/open-mmlab/mmdetection/pull/8191)? I will test with official mmdetection and our [RF-mmdetection](https://github.com/ShangHua-Gao/RF-mmdetection/tree/rfsearch/configs) as soon as possible. RF-mmdetection need some corresponding changes,...