The variables selectedClassIndex and tempSelectedClassIndex seem to have unintuitive purposes

[himanshunaidu]

One would think that tempSelectedClassIndex is used to temporarily store the value of selectedClassIndex, but its purpose is slightly different as the code suggests:

iOSPointMapper/IOSAccessAssessment/Views/AnnotationView.swift

Lines 75 to 79 in 51bd3f9

	if option == .misidentified {
	selectedClassIndex = index
	tempSelectedClassIndex = sharedImageData.segmentedIndices[index]
	isShowingClassSelectionModal = true
	}

[himanshunaidu]

Also tempSelectedClassIndex isn't actually used anywhere after the assignment.
Just need some clarity on why tempSelectedClassIndex is used.
@mariana0412

[mariana0412]

I guess I planned to use it not to allow the user to choose the option "The class annotation is misidentified" and first select a different class but then the one that was assigned by the model in the first place (the option "I agree with this class annotation should be chosen in this case).
But, as I remember, we decided not to overcomplicate this class selection.
So, if it is not implemented, this variable can be safely removed.

[himanshunaidu]

I see. Yes, that makes sense.
Let's park this issue for a while.
We'll see what we can do to take care of this confusion, since currently, both the options seem like they will cause some confusion, unless we do some UI changes.

[himanshunaidu]

The option has been (mostly) commented out, so this should not be a problem.
It would seemingly be far more convenient for the user to do take care of only one class of annotations per page of AnnotationView, without having to worry about an object belonging to some other class.

But this code, will not be completely removed for now.