Skip to content

Fix IndexError#26

Open
margaritaG wants to merge 1 commit intoScanNet:masterfrom
margaritaG:fix/fix_index_error
Open

Fix IndexError#26
margaritaG wants to merge 1 commit intoScanNet:masterfrom
margaritaG:fix/fix_index_error

Conversation

@margaritaG
Copy link
Copy Markdown

@margaritaG margaritaG commented Mar 13, 2019

There exists one execution path that results in an IndexError, this fix solves it.

Specifically, whenever we evaluate a single scan which contains a match m that both has_gt and has_pred, in which all predictions are non-matched but also all of them are ignored and not counted as FP, then y_true and consequently y_true_sorted_cumsum are empty arrays.

Then accessing y_true_sorted_cumsum[-1] results in an IndexError.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant