A couple of metrics are missing ignore_index

isaaccorley / a-change-detection-reality-check

Code and experiments for the paper, "A Change Detection Reality Check", Corley et al.

MIT License

43 stars 5 forks source link

Closed robmarkcole closed 6 months ago

robmarkcole commented 8 months ago

missing from:

isaaccorley commented 8 months ago

Good catch. While I'm recording these metrics I actually only used the classwise metrics for the results.

Other papers also report overall accuracy (OA) without ignoring the no-change class which is why I removed that.

robmarkcole commented 8 months ago

without ignoring the no-change class - is that related to ignore_index?

isaaccorley commented 8 months ago

Yeah I'm not using the ignore_index in those metrics because there are no values I'm trying to ignore.