United4Surveillance / signal-detection-tool

A tool for detection of signals in infectious disease surveillance data.
Other
8 stars 0 forks source link

[FEATURE] Investigate options to increase speed of signals computations due to pad_signals() #299

Open tinneuro opened 6 months ago

tinneuro commented 6 months ago

Is your feature request related to a problem? Please describe.

We want to increase speed in our app for the signals computation especially when the user selects stratifications with a lot of strata. When I wrote the function pad_signals() I realised that this has the potential to be a source of speed loss as we are basically running the get_signals() another time to pad the signals.

Describe the Solution You'd Like

Investigate whether running the get_signals() once with the full timeperiod including the weeks for padding is faster than running it twice once for only the number_of_weeks and once for the padding. If so improve and change the way we generate the padded signals so that speed is increased.

Describe Alternatives

A clear and concise description of any alternative solutions or features you've considered.

Additional Context

Add any other context or screenshots about the feature request here.