PaddlePaddle / PaddleRS

Awesome Remote Sensing Toolkit based on PaddlePaddle.
Apache License 2.0
381 stars 90 forks source link

Revert "update batch evaluate" #171

Closed Bobholamovic closed 1 year ago

Bobholamovic commented 1 year ago

Reverts PaddlePaddle/PaddleRS#154 This is because the updates in #154 breaks the basic inference functionality of PaddleRS.

huilin16 commented 1 year ago

好的,我周日再检查下,之前测试是没有问题的,我在测下是什么问题

Bobholamovic commented 1 year ago

好的,我周日再检查下,之前测试是没有问题的,我在测下是什么问题

嗯嗯,辛苦啦~ 好像主要是postprocess里面加了一句concat,导致输出的格式和以前不一致了

huilin16 commented 1 year ago

我测试后发现,slide_predict的预测都没问题,predict的预测,当batchsize=1时,返回的是dict,但batchsize>1时,输入的img_file是list,因此输出的结果也是list,这是与slide_predict的接口相匹配的,如果把predict的输出结果全改成dict,slide_predict就会出错

Bobholamovic commented 1 year ago

请问确认分割和变化检测任务的slider_predict在batch_size > 1的情况下可以正常工作嘛?好像修改之后后处理返回的输出与我们在docstring和文档里说明的不太一样哦?

huilin16 commented 1 year ago

slider_predict没有返回值吧,只有输出的预测结果,我测试的slide_predict的预测都没问题

huilin16 commented 1 year ago

有问题的是predict,predict的输出结果是个list,因为batchsize>1时,输入的img_file是list

huilin16 commented 1 year ago

seg1 seg2 cd1 cd2

Bobholamovic commented 1 year ago

好吧,看起来好像和我上次测试的时候发现的结论不太一样😂 也有可能是我那时候搞错了~ 辛苦试试看使用revert前的代码能不能通过单元测试,如果可以通过的话,可以重提一个PR revert我revert的那个commit~