[Fix] Fix test error when using --show-dir

[zergzzlun]

Motivation

Fix #1073
Test error mentioned in #1073 is caused by pre_eval() changing the segmentation result.

Modification

Changed results from pre_eval() from result to pre_eval_results.

[CLAassistant]

All committers have signed the CLA.

[Junjun2016]

Hi @zergzzlun
Thanks for your nice fix.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.54%. Comparing base (91cbe06) to head (61ddc29).
Report is 287 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1091      +/-   ##
==========================================
- Coverage   89.57%   89.54%   -0.04%     
==========================================
  Files         120      122       +2     
  Lines        6717     6932     +215     
  Branches     1122     1156      +34     
==========================================
+ Hits         6017     6207     +190     
- Misses        496      519      +23     
- Partials      204      206       +2     

Flag	Coverage Δ
unittests	89.54% <100.00%> (-0.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[MengzhangLI]

It works well on my local environemt. Fix the bug.

Before:

After:

[xvjiarui]

Hi @zergzzlun

Thanks for your contribution. I just created a PR https://github.com/zergzzlun/mmsegmentation/pull/1 in your repo (since I couldn't push to your repo directly).

Could you please have a look on that?

[MengzhangLI]

Hi, @zergzzlun .

Please have a look at pr in your forked repo. If you approve it, we would merge this pr as soon as possible.

Thanks again for your contribution.

[zergzzlun]

Hi team.
Thanks for your kind words. zergzzlun#1 has been merged.

[MengzhangLI]

Hi team. Thanks for you kind words. zergzzlun#1 has been merged.

Hi, could you have a try on your new commit code with/witout --show-dir command? Thank you.

[zergzzlun]

Hi team. Thanks for you kind words. zergzzlun#1 has been merged.

Hi, could you have a try on your new commit code with/witout --show-dir command? Thank you.

Tested it. The new version is fine.

[Junjun2016]

LGTM

[haims-hblab]

It works perfectly.

[OpenMMLab-Assistant003]

Hi @zergzlun！First of all, we want to express our gratitude for your significant PR in the MMSegmentation project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.

We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA

If you have WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤ @zergzlun