Optimisation of chart data django code may help to speed up calculation time.
No description provided.
Comments (18)
-
reporter -
reporter - changed status to resolved
I don't think there is anything further that I can do with this now.
-
reporter I have made some further optimisations during my work with issue
#332. -
reporter Further optimisation of radiographic chart data calculations. References issue
#217→ <<cset 30915f484e19>>
-
reporter Further optimisation of radiographic chart data calculations. References issue
#217→ <<cset 30915f484e19>>
-
reporter - changed status to open
changed how data for acquistion-based charts is obtained. Upside: much, much faster. Downside: can't exclude Constant Angle Acquisition data from CT chart data at the moment. References issue
#390and reopens issue#217→ <<cset 03082d4d9e03>>
-
reporter Revised average chart data calculations so that constant angle acquisitions can be ignored. References issue
#217→ <<cset 7c2ea1aee5ef>>
-
reporter Corrected a couple of omissions. Clarified the True parameter when passing to the chart calculations. References issue
#217→ <<cset c08cab4e4700>>
-
changed how data for acquistion-based charts is obtained. Upside: much, much faster. Downside: can't exclude Constant Angle Acquisition data from CT chart data at the moment. References issue
#390and reopens issue#217→ <<cset 03082d4d9e03>>
-
Revised average chart data calculations so that constant angle acquisitions can be ignored. References issue
#217→ <<cset 7c2ea1aee5ef>>
-
Corrected a couple of omissions. Clarified the True parameter when passing to the chart calculations. References issue
#217→ <<cset c08cab4e4700>>
-
reporter Make calculation and plotting of histograms optional. Without these the data will be quicker to calculate.
-
reporter Made calculation of histogram data optional. Huge improvement in calculation time when switched off. References issue
#217→ <<cset 753c9ab7eaa7>>
-
reporter f.qs only results in duplicate studies if an acquisition-level filter value is present. So for CT, provided that the user has not filtered on acquisition protocol then f.qs can be used without any duplicates.
-
reporter Use fast query method for study plot data when there is no acquisition protocol filter specified (there is no issue with duplicates for this situation). Use slower method when there is an acquisition protocol filter, otherwise there would be duplicates. Removed display of average choice for mammo charts as there is no chart that this is used for at the moment. References issue
#217and issue#390→ <<cset 425fd3ed5d9f>>
-
reporter Fixed problem when
f.form.data['acquisition_protocol']
isn't defined when a user first goes to a page. References issue#217→ <<cset 2db52bc32be4>>
-
reporter - changed status to resolved
This is as optimised as it is going to get for the time being.
-
reporter Fixing chart sorting by frequency or value that I accidentally broke a few days ago in commit 753c9ab. Added code so that the required values are calculated whether histograms are enabled or not. References issue
#217→ <<cset e532a8ae620a>>
- Log in to comment
Made some modifications to speed up chart calculations. Only calculations that are required by the selected charts are now carried out. References issue 217.
→ <<cset e93b8c14e7b5>>