Visualizer show classification and segmentation

[alexriedel1]

This would be a possible fix to #175 (comment)

Alternatively, the classification image could always be added as it provides useful insights in what the model predicted.

[samet-akcay]

Thanks for the pr, would you be able to resolve the merge conflicts so we could run the CI?

[alexriedel1]

Thanks for the pr, would you be able to resolve the merge conflicts so we could run the CI?

Ah right! Just want to note that after merging this PR #181 the proposed solution needs to be updated..

[samet-akcay]

@alexriedel1, the CI has been fixed now. Could you please update the PR by pulling the latest development? Thanks!

[alexriedel1]

@alexriedel1, the CI has been fixed now. Could you please update the PR by pulling the latest development? Thanks!

done

[alexriedel1]

I merged the development branch and fixed some styling errors

[samet-akcay]

@alexriedel1, we've run the CI. There are two tests that failed unexpectedly. We'll have a look why this is the case, and let you know. I don't expect you to look the tests:)

Here is the CI log showing why it failed:
CI-PR-178.log

[alexriedel1]

@alexriedel1, we've run the CI. There are two tests that failed unexpectedly. We'll have a look why this is the case, and let you know. I don't expect you to look the tests:)

Here is the CI log showing why it failed: CI-PR-178.log

It's because the class VisualizerCallback now has to be initialized with a task. I can check that and fix it if you want.
/Edit: You can check my solution to this now. Another way would be to set a default task value in the VisualizerCallback but I guess this is a bit counter intuitive.

[ashwinvaidya17]

Thanks for the efforts!

[djdameln]

Thanks for another nice contribution. I just have some minor comments