Skip to content
This repository has been archived by the owner on Feb 2, 2022. It is now read-only.

V0.7.5dev #64

Merged
merged 45 commits into from
Apr 9, 2020
Merged

V0.7.5dev #64

merged 45 commits into from
Apr 9, 2020

Conversation

cutright
Copy link
Owner

@cutright cutright commented Apr 8, 2020

v0.7.5 (TBD)

  • [Import] Option to import DVH stored in DICOM RT-Dose rather than calculate it. NOTE: Currently the source of the DVH
    is not stored in the database.
  • [Import] Fixed bug that caused datetime parsing to fail if DICOM timestamp had fractional seconds
    (avoid the "unconverted data remains" error).
  • [Import] Option to turn off automatic dose summation: #57
  • [Import] Bug fix: #59
  • [Import] SQLite import_time_stamp now actually includes time, and is in your local timezone
  • [Import] SoftwareVersions and Deferred read error fix: #36
  • [Export] New figure export implementation in tool bar, supports SVG and HTML. Note that SVG export requires phantomJS to be installed, which is not a python library.

SVG export requires both a new python library (selenium) and phantomJS. On macOS, I had success with brew install npm then npm install phantomjs. Or alternatively, conda install phantomjs is an option. I'm not sure I'll be able to test on Windows without admin privileges.

cutright added 30 commits March 14, 2020 11:37
@cutright
begin dev for DVH Constraints: #53
a6890fa
@cutright
DIRECTORIES dict and PROTOCOL_DIR added to dvha.paths
e8374a7
@cutright
DIRECTORIES dict and PROTOCOL_DIR added to dvha.paths, copy protocols…
fbd33ea 
… to user dir if missing
@cutright
merge dvh.py and protocols.py so Constraint can use endpoint calc fun…
1afc8f5 
…ctions
@cutright
begin dev for protocol editor window (currently non-functional)
bc9edd0
@cutright
suppress fuzzywuzzy's python-Levenshtein install warning
3258948
@cutright
remove unnecessary About & PythonLibraries launch functions
3a1e82d
@cutright
update protocol table with combobox changes
fef1b60
@cutright
ProtocolEditor delete functionality, add constraint dlg framework
d120598
@cutright
cosmetic tweaks
26ef51d
@cutright
added auto kwarg for set_column_widths to use wx.LIST_AUTOSIZE
8332c4c
@cutright
begin dev of showing TG263 spreadsheet on ROI Map
876c080
@cutright
export plots to .svg. This branch requires selenium and phantomjs.
35c973a 
#56
@cutright
Database Editor: Fix for #55
6f919d5
@cutright
Add filters for TG-263 table in ROI Map Editor, better auto column wi…
45e1012 
…dths, sorting
@cutright
Add protocol constraint progress
4724e7b
@cutright
Allow DVH from DICOM RT-Dose, rather than recalculating
7371315 
dicompyler-core has a from_dicom_dvh method to import DVH from DICOM, enable this
in User Settings. Future versions will add a column to the DVHs table to keep track
of the DVH source. Feature added with SRS in mind since 1mm dose grid DVH calculations
can take a long time. #57
@cutright
Dose summation is now optional
cd92c04 
If multiple dose grids are found, user can choose to import plan DVHs instead of a composite DVH.
If plan DVHs imported, study_instance_uid will be appended by _N where N is the nth plan in set.
#57
@cutright
Dicom dose sum is over a study, not a patient
6506523 
#57
@cutright
Fixed bug where DICOM timestamps failed to parse if fractional second…
5569935 
…s were used
@cutright
begin dev for ptv assignment per plan when disabling auto-summing of …
a8e2931 
…dose

#57
@cutright
fix: if len of new data is different than previous, sorting may crash
604bab6
@cutright
implement plan PTV assignments (multi-plan/single Structure, no dose …
452b181 
…summation)

#57
@cutright
SQLite import_time_stamp fix. And roi name/type over-rides not applie…
5d45900 
…d bug fix

#59
@cutright
Merge pull request #63 from cutright/svg_test
ba9b88d 
export plots to .svg. This branch requires selenium and phantomjs.
@cutright
Simplify plot export menu. options not yet functional but SVG & HTML …
ac8920b 
…exports work

#56
@cutright
Export plot options functional for SVG only
8fbd89a 
#56
Should probably store figure properties per plot type instead of one size fits all
@cutright
If text field is empty, use current figure attr value, FileDialog ini…
7b98dfd 
…tDir property

#56
@cutright
Fix for issues when SoftwareVersions is stored not in RT Plan and is …
0272cce 
…not a list

#36
@cutright
dicompyler DicomParser may lead to a a deferred read error for some f…
2506319 
…orce opened dicom files

OSError: Deferred read -- original filename not stored. Cannot re-open
#36
@lgtm-com
Copy link

lgtm-com bot commented Apr 9, 2020

This pull request introduces 1 alert when merging 5fc8535 into faa999e - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Apr 9, 2020

This pull request introduces 2 alerts when merging 18ac0cc into faa999e - view on LGTM.com

new alerts:

  • 1 for Unused import
  • 1 for Wrong number of arguments in a class instantiation

@cutright cutright merged commit 2131a92 into master Apr 9, 2020
@cutright cutright deleted the v0.7.5dev branch April 9, 2020 22:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

TPSs that work with DVHA Edited ROI Name on Import Screen Not Propagating to Import (v0.7.4)
1 participant
@cutright