Adding Skip-Button to PTV Assignment Window

[Kiragroh]

Hi Dan,

in one cohort of mine, I have 270 plans for the same StudyInstanceUID and StructureSet. There is only one PTV per Plan and no PTV-assignment necessary (at other times much appreciated).

Because I do not like to click 270 times to come to the Finish-Button I added a 'Skip to the end'-Button. The important new button-function is here:

def on_skip(self, *evt):
    self.update_data(len(self.uids)-self.current_index)

Maybe interesting to add to the next release but maybe is can do more harm for standard imports than help in more special cases.

[cutright]

I should probably automatically exclude any patients in that list that have no PTVs to assign, and then don't show the window at all if the list is empty.

Honestly, I've never used this feature clinically. So feedback is very welcome.

[Kiragroh]

This would be nice and solve most cases.

Additional thought: Maybe you can auto-assign a ptv to included-list where only one PTV would be listed in the ignored-panel

[cutright]

Does this commit above work? I don't have a set of data to easily test this. I added the following lines every time update_data is called:

DVH-Analytics/dvha/models/import_dicom.py

Lines 2580 to 2587 in 2b97ed2

	# Auto skip if < 2 PTVs
	ptv_count = len(self.ptvs[self.study_uid])
	if ptv_count == 0:
	self.on_next()
	elif ptv_count == 1:
	self.ptvs[self.plan_uid] = [self.ptvs[self.study_uid]]
	self.update_data()
	self.on_next()

Might need a dialog window to explain what's happening though? I'm also unsure if the self.update_data() is necessary.

[cutright]

Correction, had a type issue:

DVH-Analytics/dvha/models/import_dicom.py

Lines 2580 to 2587 in 59b6bf4

	# Auto skip if < 2 PTVs
	ptv_count = len(self.ptvs[self.study_uid])
	if ptv_count < 2:
	if ptv_count == 1:
	ptvs = self.ptvs[self.study_uid]
	self.ptvs[self.plan_uid] = self.ptvs[self.plan_uid].union(ptvs)
	self.update_data()
	self.on_next()

[cutright]

I think I've got it, came across a few unrelated issues along the way.

If all of your plans can auto assign PTVs, you'll see the window blip in and out before you can do anything. I'd prefer that it didn't show at all, but I think that'll require a lot more thought (although maybe not much code?). If any of the plans have multiple PTVs to pick from, the window should still let you assign and go back and forth through only the plans with multiple PTVs available.

I also came across periodic issues ignoring uncategorized ROIs... not sure why yet.
RuntimeError: dictionary changed size during iteration