Action #1300
Add pre-computation cache to GModelSpatialRadialGauss
| Status: | Closed | Start date: | 07/26/2014 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assigned To: |  Knödlseder Jürgen | % Done: | 100% | |
| Category: | - | |||
| Target version: | 2.0.0 | |||
| Duration: |
Description
I recognized that GModelSpatialRadialGauss has no pre-computation cache. A bunch of multiplications and one division can be saved when all sigma related stuff is cached, which then would be only recomputed when the sigma value changes.
Recurrence
No recurrence.
History
#1
Updated by Knödlseder Jürgen about 10 years ago
- Description updated (diff)
#2
Updated by Knödlseder Jürgen over 5 years ago
- Target version set to 1.7.0
#3
Updated by Knödlseder Jürgen over 4 years ago
Moved issue to next release.
#4
Updated by Knödlseder Jürgen over 4 years ago
- Target version changed from 1.7.0 to 2.0.0
#5
Updated by Knödlseder Jürgen about 4 years ago
- Status changed from New to In Progress
- Assigned To set to Knödlseder Jürgen
- % Done changed from 0 to 50
I implemented caching. The tables below compare the performance before and after implementation of the caching. The speed-up is negligible.
Before caching:| Model | CPU_r | Iter_r | logL_r | CPU_n | Iter_n | logL_n | Speed-up | Comments |
| Gaussian | 307.80 | 2 | 118106.615 | 16.55 | 2 | 118106.615 | 18.6 | identical results |
| Model | CPU_r | Iter_r | logL_r | CPU_n | Iter_n | logL_n | Speed-up | Comments |
| Gaussian | 307.80 | 2 | 118106.615 | 16.28 | 2 | 118106.615 | 18.9 | identical results |
#6
Updated by Knödlseder Jürgen about 4 years ago
- Status changed from In Progress to Closed
- % Done changed from 50 to 100
Code merged into devel.