develop
Author | Commit | Message | Date | Builds |
---|---|---|---|---|
5fd625c
M
|
Merged in issue886AddressSonarCloudBugs (pull request #440)
Fixes #886
Approved-by: Ed McDonagh
Thanks David. I'll leave the translate flags for aria tags for now.
|
|
||
Updating changes file [skip ci]. Refs issue
|
|
|||
Correcting error where I used colspan instead of scope [skip ci]. Refs issue
|
|
|||
Addressing SonarCloud issues for the remaining flies with at least 10 bugs [skip ci]. Refs issue
|
|
|||
Addressing the new SonarCloud issues [skip ci]. Refs issue
|
|
|||
Replaced use of <row> with <tr>. Split some table headings onto separate lines [skip ci]. Refs issue
|
|
|||
Added table aria-describedby and th scope for more files [skip ci]. Refs issue
|
|
|||
Added table aria-describedby and th scope in dicomsummary.html [skip ci]. Refs issue
|
|
|||
Added table aria-describedby and th scope in review_summary_list.html [skip ci]. Refs issue
|
|
|||
Replaced table captions with aria-describedby. Avoids having to hide anything [skip ci]. Refs issue
|
|
|||
Hiding captions as their text is already in the section headings. Hopefully this still satisfies sonarcloud and screen readers [skip ci]. Refs issue
|
|
|||
Adding captions to tables and scope to table headings in the detailed view templates [skip ci]. Refs issue
|
|
|||
Fixing bug (which did no harm) in skin dose map JavaScript. Refs issue
|
|
|||
55e24e0
M
|
Merged in issue861AddMammoFilters (pull request #439)
Issue861AddMammoFilters
Approved-by: Ed McDonagh
Fixes issue #861
|
|
||
Trying to disable too-many-lines again [skip ci]. Refs issue
|
|
|||
Trying to disable too-many-lines for the whole of the mod_filters file [skip ci]. Refs issue
|
|
|||
Trying to disable line-too-long for the whole of the mod_filters file [skip ci]. Refs issue
|
|
|||
Ran Black and added two pylint disable=line-too-long comments [skip ci]. Refs issue
|
|
|||
Added tests for the new mammography filters. Refs issue
|
|
|||
Changing order of filters so that `Compressed breast thickness` and `Exposure control mode` appear higher up the list. Refs issue
|
|
|||
Updating changes file. Refs issue
|
|
|||
Moved `Exposure control mode comment` to a column at the end of each row. Made the table sortable. Refs issue
|
|
|||
Split display of `Laterality` and `View code` in mammography detail view. Refs issue
|
|
|||
Added display of exposure control mode in mammography detail view (makes sense to display this now that a user can filter on the value). Altered radiographic title to read `Exposure control mode` rather than `Control mode` to better reflect what is being displayed and to make it consistent with the mammography page. Refs issue
|
|
|||
7614861
M
|
Merged in issue810openskinhead (pull request #390)
Fixes #810
Approved-by: David Platten
Approved-by: Ed McDonagh
|
|
||
01b626e
M
|
Merge branch 'develop' into issue810openskinhead
|
|
||
Removing packaging requirement, as per Wens comment on PR. Refs
|
|
|||
Modified compressed breast thickness range label [skip ci]. Refs issue
|
|
|||
Modified mammography filters for compressed breast thickness because the original didn't work properly. This one does. Refs issue
|
|
|||
Added mammography filter for acquisition exposure control mode (manual, automatic etc). Refs issue
|
|
|||
Added mammography filters for compressed breast thickness. Refs issue
|
|
|||
Added mammography filter for view code meaning. Refs issue
|
|
|||
704fb25
M
|
Merged in issue867plotlyChartDocs (pull request #438)
Added documentation of `Download csv` button for average bar charts and frequency charts [skip ci]. Refs issue #867
|
|
||
Added documentation of `Download csv` button for average bar charts and frequency charts [skip ci]. Refs issue
|
|
|||
d211d84
M
|
Merged in issue867chartDocFormatting (pull request #437)
Forcing a line break after each example chart [skip ci]. Refs issue #867
|
|
||
Forcing a line break after each example chart [skip ci]. Refs issue
|
|
|||
8597e2d
M
|
Merged in issue870improveDataFrameCreationMemoryEfficiency (pull request #436)
Issue870improveDataFrameCreationMemoryEfficiency
Approved-by: Ed McDonagh
Fixes issue #870
Refs issue #881
|
|
||
82e4a79
M
|
Merged in issue860removeStoreSCP (pull request #434)
Fixes #860
|
|
||
Black with v20.8b1. Refs
|
|
|||
Combined the two whitespace options into one. Refs issue
|
|
|||
Removed the custom ordering on the query set before creation of chart DataFrame. This custom ordering isn't needed by the chart routines (it is used to order the summary view tables in the order specified by the user), and slows down the DataFrame creation. In some of my tests this reduces the DataFrame creation time from 10 s to 7 s. Refs issue
|
|
|||
Trying to appease Codacy. Refs issue
|
|
|||
Ran Black. Refs issue
|
|
|||
Commented DataFrame creation more thoroughly. Removed loop through names. Refs issue
|
|
|||
44bef5b
M
|
Merged in issue867docsForPlotlyCharts (pull request #435)
Issue867docsForPlotlyCharts
Approved-by: Ed McDonagh
Fixes issue #867 and fixes issue #881
|
|
||
Updating Plotly JavaScript to 1.58.2 to go with Plotly Python package version 4.14.1. Refs issue
|
|
|||
Reducing table width to prevent scroll bars [skip ci]. Refs issue
|
|
|||
Reducing table width to prevent scroll bars [skip ci]. Refs issue
|
|
|||
Changing width back to 1000px; will tinker with tables to prevent scrollbars in a bit. Refs issue
|
|
|||
Addressing Codacy issues. Refs issue
|
|
|||
Removed the rainbow and jet colour scale options. Refs issue
|
|
|||
Added a couple of anchors to improve linking within the document [skip ci]. Refs issue
|
|
|||
Replaced | with |br| raw html for line breaks in tables [skip ci]. Refs issue
|
|
|||
Previous commit wasn't quite right. This should do the job [skip ci]. Refs issue
|
|
|||
Does this force a line break? [skip ci]. Refs issue
|
|
|||
Fiddling with table layout (hopefully for the final time) [skip ci]. Refs issue
|
|
|||
Trying to keep first column text from splitting into two rows [skip ci]. Refs issue
|
|
|||
Re-wording description of over-time example text. Trying to keep first column text from splitting into two rows [skip ci]. Refs issue
|
|
|||
Changed over-time example to CT, and added a chart grouped by system name as well as by requested procedure name [skip ci]. Refs issue
|
|
|||
Added example frequency chart grouped by x-ray system name (forgot to add the file last time) [skip ci]. Refs issue
|
|
|||
Added example frequency chart grouped by x-ray system name [skip ci]. Refs issue
|
|
|||
Changing custom css method so it follows the recommended method (https://docs.readthedocs.io/en/latest/guides/adding-custom-css.html). Refs issue
|
|
|||
Added text describing each chart type more thoroughly. Need to include example of fluoro split by physician, and a chart where the grouping is switched. Refs issue
|
|
|||
Changing width of doc page to 1200px from 1000px to see if it removes the scroll bars on one of the tables. Refs issue
|
|
|||
Adding some description to the workload chart. Ed - does this clarify things for you? [skip ci]. Refs issue
|
|
|||
Changing width of doc page to 1000px from full width. Will still be wider than the default 800px. Refs issue
|
|
|||
|
||||
7cebf5a
M
|
Merge branch 'issue867docsForPlotlyCharts' of https://bitbucket.org/openrem/openrem into issue867docsForPlotlyCharts
|
|
||
Updated changes file [skip ci]. Refs issue
|
|
|||
b8901be
M
|
Merged develop into issue867docsForPlotlyCharts
|
|
||
|
||||
Added some themed charts [skip ci]. Refs issue
|
|
|||
Added the remaining colourmap swatches. Refs issue
|
|
|||
Added more to the chart docs. Re-ordered chart option entries for consistency. Refs issue
|
|
|||
Added stylesheet to try and force max width of doc pages to be larger. Following the suggestion here: https://stackoverflow.com/questions/23211695/modifying-content-width-of-the-sphinx-theme-read-the-docs [skip ci]. Refs issue
|
|
|||
b3cac9c
M
|
Merge remote-tracking branch 'origin/issue810openskinhead' into issue810openskinhead
|
|
||
Moving skin map version from separate line. Refs
|
|
|||
a8bcc4d
M
|
Merge branch 'develop' into issue810openskinhead
# Conflicts:
# openrem/remapp/views.py
|
|
||
Fixing Codacy catches. Refs
|
|
|||
a6db940
M
|
Merge branch 'develop' into issue860removeStoreSCP
|
|
||
c759ac1
M
|
Merged in issue856dropAcqFilters (pull request #430)
Fixes #856
|
|
||
Adding ref
|
|
|||
Tidying up status message display, plus link to summary page. Refs
|
|
|||
Added remaining chart examples to the docs. Fixed typo in binned statistic file name - it was the same as the mammo scatter plot. Refs issue
|
|
|||
Added some example charts to the docs. Added line of code to remove any trailing whitespace at the end of name values in the chart code. Refs issue
|
|
|||
Added new settings variable to control the creation of the Plotly html chart files. Refs issue
|
|
|||
Added method to save Plotly charts as an HTML DIV in the MEDIA_ROOT folder. I need this to be able to produce charts for the documentation. It is deactivated for all users, and can only be activated by modifying the source code. Also renamed file_name to filename in some chart code for consistency. Refs issue
|
|
|||
Removing stop/start buttons. Removing future import. Refs
|
|
|||
Testing to see if embedding an example Plotly chart in the docs works. Refs issue
|
|
|||
Updating doc to address very wide tables [skip ci]. Refs issue
|
|
|||
Written initial Plotly chart doc skeleton [skip ci]. Refs issue
|
|
|||
2cdb8cb
M
|
Merged in issue869docStringsForNewChartCode (pull request #433)
Issue869docStringsForNewChartCode
Approved-by: Ed McDonagh
Fixes issue #869
|
|
||
Fixing broken drawing of head part of skin dose map. Apologies - it was my JavaScript re-write that broke this. Refs issue
|
|
|||
Removing old_div that depended on future. Still doesn't really ref
|
|
|||
Removing future from project, doesn't really ref
|
|
|||
0882277
M
|
Merge branch 'develop' into issue860removeStoreSCP
|
|
||
Removing controlled and keep alive database values and functions for ref
|
|
|||
Switched sign on head 'lid' rotation so that the top of the head is opaque (and blue). Also using phantomHeadRadius to set the radius of this lid as it is clearer. Refs issue
|
|
|||
Small adjustment to avoid exceeding recommended line length (Codacy) [skip ci]. Refs issue
|
|
|||
Adjusting layout of docstrings to try and fix formatting issues [skip ci]. Refs issue
|
|
|||
Added the last docstrings for the chart code [skip ci]. Refs issue
|
|
|||
Added more docstrings [skip ci]. Refs issue
|
|
|||
Adding more docstrings for chart code [skip ci]. Refs issue
|
|
|||
Added docstring for box plots [skip ci]. Refs issue
|
|
|||
75a2fc0
M
|
Merged develop into issue869docStringsForNewChartCode
|
|
||
7e6b3d3
M
|
Merged in issue879boxplotPhysicianSorting (pull request #432)
Issue879boxplotPhysicianSorting
Approved-by: Ed McDonagh
Fixes issue #879
|
|
||
Updating changes file [skip ci]. Refs issue
|
|
|||
Fixed sorting of boxplots (and bar charts) for fluoroscopy charts when split by physician. Also fixed bug where sorted_study_categories were being used for request charts. Refs issue
|
|
|||
f677df8
M
|
Merged develop into issue869docStringsForNewChartCode
|
|
||
Fixing issue where exclusion of lots of missing data causes height of each chart subplot to be very large. Calculating the chart height after the exclusion fixes this. Refs issue
|
|
|||
Reordering entries [skip ci]. Refs issue
|
|
|||
Trying again [skip ci]. Refs issue
|
|
|||
Trying again [skip ci]. Refs issue
|
|
|||
Trying to itemise the parameter dictionary items [skip ci]. Refs issue
|
|
|||
Trying to itemise the parameter dictionary items [skip ci]. Refs issue
|
|
|||
Added some more docstrings [skip ci]. Refs issue
|
|
|||
Initial commit of ref
|
|
|||
fe6de0a
M
|
Merge branch 'develop' into issue856dropAcqFilters
# Conflicts:
# docs/changes.rst
|
|
||
da63eae
M
|
Merged in issue863weightunitsdelete (pull request #431)
Fixes #863
|
|
||
Added ref
|
|
|||
Removing height and weight from delete study summary information. Refs
|
|
|||
Running black on ref
|
|
|||
Added ref
|
|
|||
|
||||
Added very basic test for ref
|
|
|||
0685d46
M
|
Merged develop into issue869docStringsForNewChartCode
|
|
||
Removing unicode u markers in CT filters tests, refs
|
|
|||
Fixing existing CT export tests for ref
|
|
|||
Removed acquisition type choices. Refs
|
|
|||
Added "more than 0" to CT acquisition type filters. Refs
|
|
|||
Translated using Weblate (Italian)
Currently translated at 100.0% (28 of 28 strings)
Translation: OpenREM/RTD Installation Docker env config
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-installation-docker-env-config/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 21.3% (19 of 89 strings)
Translation: OpenREM/RTD Installation Linux non-Docker
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-installation-linux-non-docker/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 100.0% (28 of 28 strings)
Translation: OpenREM/RTD Installation Docker env config
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-installation-docker-env-config/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 21.3% (19 of 89 strings)
Translation: OpenREM/RTD Installation Linux non-Docker
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-installation-linux-non-docker/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 21.3% (19 of 89 strings)
Translation: OpenREM/RTD Installation Linux non-Docker
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-installation-linux-non-docker/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 100.0% (28 of 28 strings)
Translation: OpenREM/RTD Installation Docker env config
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-installation-docker-env-config/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 100.0% (66 of 66 strings)
Translation: OpenREM/Web Interface
Translate-URL: https://hosted.weblate.org/projects/openrem/web-interface/it/
|
|
|||
Updating translations, adding in Italian. Refs #855
|
|
|||
Translated using Weblate (Italian)
Currently translated at 15.7% (14 of 89 strings)
Translation: OpenREM/RTD Installation Linux non-Docker
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-installation-linux-non-docker/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 100.0% (10 of 10 strings)
Translation: OpenREM/RTD Index
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-index/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 100.0% (66 of 66 strings)
Translation: OpenREM/Web Interface
Translate-URL: https://hosted.weblate.org/projects/openrem/web-interface/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 100.0% (18 of 18 strings)
Translation: OpenREM/RTD Installation
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-installation/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 80.0% (8 of 10 strings)
Translation: OpenREM/RTD Index
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-index/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 80.0% (8 of 10 strings)
Translation: OpenREM/RTD Index
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-index/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 15.7% (14 of 89 strings)
Translation: OpenREM/RTD Installation Linux non-Docker
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-installation-linux-non-docker/it/
|
|
|||
Added translation using Weblate (Italian)
|
|
|||
Added translation using Weblate (Italian)
|
|
|||
Translated using Weblate (Italian)
Currently translated at 30.0% (3 of 10 strings)
Translation: OpenREM/RTD Index
Translate-URL: https://hosted.weblate.org/projects/openrem/rtd-index/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 100.0% (66 of 66 strings)
Translation: OpenREM/Web Interface
Translate-URL: https://hosted.weblate.org/projects/openrem/web-interface/it/
|
|
|||
Translated using Weblate (Italian)
Currently translated at 100.0% (66 of 66 strings)
Translation: OpenREM/Web Interface
Translate-URL: https://hosted.weblate.org/projects/openrem/web-interface/it/
|
|
|||
Added translation using Weblate (Italian)
|
|
|||
Translated using Weblate (Italian)
Currently translated at 4.5% (3 of 66 strings)
Translation: OpenREM/Web Interface
Translate-URL: https://hosted.weblate.org/projects/openrem/web-interface/it/
|
|
|||
Added translation using Weblate (Italian)
|
|
|||
Translated using Weblate (German)
Currently translated at 100.0% (66 of 66 strings)
Translation: OpenREM/Web Interface
Translate-URL: https://hosted.weblate.org/projects/openrem/web-interface/de/
|
|
|||
Translated using Weblate (Spanish (Mexico))
Currently translated at 100.0% (66 of 66 strings)
Translation: OpenREM/Web Interface
Translate-URL: https://hosted.weblate.org/projects/openrem/web-interface/es_MX/
|
|
|||
547bade
M
|
Merged in issue866MultiValCharSet (pull request #429)
Fixes #866
|
|
||
Codacy - might satisfy inconsistent-return-statements. Refs #866
|
|
|||
Codacy - switching imports around. Refs #866
|
|
|||
Codacy - too many local variables. Refs #866
|
|
|||
Adding / adjusting docstrings [skip ci]. Refs issue #869
|
|
|||
Added some docstrings [skip ci]. Refs issue #869
|
|
|||
Removing redundant safe_strings function. Refs #866
|
|
|||
Added ref #866 to changes.
|
|
|||
Requesting pylint not to complain about import order for modles imports. Refs #866.
|
|
|||
Running black on ref #866.
|
|
|||
Removed SpecificCharacterSet variables from ct_philips. Moved imports to top. Refs #866.
|
|
|||
Removed SpecificCharacterSet variables from dx. Moved imports to top. Refs #866.
|
|
|||
Sorted out import order for extract_common whilst working on ref #866.
|
|
|||
Refactored patient_attribute_modules into extract_common, tidying up whilst working on ref #866.
|
|
|||
Added MultiValue SpecificCharacterString and PerformingPhysicianName with escape character, and test of import. Refs #866.
PersonName is copied from pydicom examples which are taken from David Clunie's examples.
|
|
|||
dfc904a
M
|
Merged in issue810openskinhead_rfSkinDoseMap3dObject (pull request #423)
Trying to reduce complexity of this.draw method. Refs issue #810
Approved-by: David Platten <dplatten@gmail.com>
|
|
||
Adding the charts doc to the menu [skip ci]. Refs issue #869
|
|
|||
Adding rst file for auto-generation of chart code documentation from the doc strings. Refs issue #869
|
|
|||
997dc13
M
|
Merged in issue869docStringsForNewChartCode (pull request #428)
Issue869docStringsForNewChartCode
Approved-by: David Platten <dplatten@gmail.com>
Approved-by: Ed McDonagh <ed@mcdonagh.org.uk>
Fixes issue #871
Fixes issue #872
|
|
||
Addressing Codacy issues with the new doc strings [skip ci]. Refs issue #869
|
|
|||
Updated changes file [skip ci]. Refs issue #872
|
|
|||
Added to the doc strings. Refs issue #869
|
|
|||
Added code to create fluro charts of average DAP over time split per physician and also histograms split by physician. Refs issue #872
|
|
|||
Removed the use of safe_strings and passing of the character set from rdsr. Refs #866.
|
|
|||
Updated changes files [skip ci]. Refs issue #869 and issue #871
|
|
|||
Added brief doc strings for the remaining chart functions. Refs issue #869
|
|
|||
Refs issue #869 and issue #871
|
|
|||
1e65e03
M
|
Merged develop into issue869docStringsForNewChartCode
|
|
||
2f90276
M
|
Merged in issue870improveDataFrameMemoryEfficiency (pull request #426)
Using values_list rather than values to read Django queryset into the DataFrame. This reduces the memory requirement by around 50 %. It is also faster (3 s vs 5 s for 600,000 radiographic studies when plotting the workload chart). https://stackoverflow.co
Approved-by: Ed McDonagh <ed@mcdonagh.org.uk>
Fixes issue #870
|
|
||
Matching the requirements files. Updating spacing for some charts. Simplifying weekday dataframe creation. Refs issue #870
|