Optimisation of chart data django code may help to speed up calculation time.

Issue #217 resolved
David Platten created an issue

No description provided.

Comments (18)

  1. David Platten reporter

    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>>

  2. David Platten 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 #390 and reopens issue #217

    → <<cset 03082d4d9e03>>

  3. Ed McDonagh

    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 #390 and reopens issue #217

    → <<cset 03082d4d9e03>>

  4. David Platten reporter

    Make calculation and plotting of histograms optional. Without these the data will be quicker to calculate.

  5. David Platten 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.

  6. David Platten 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 #217 and issue #390

    → <<cset 425fd3ed5d9f>>

  7. David Platten 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>>

  8. Log in to comment