Cross-Correlation performance - Githubissues

swiss-seismological-service / scdetect

A computationally efficient earthquake detection module for SeisComP

https://scdetect.readthedocs.io

GNU Affero General Public License v3.0

15 stars 6 forks source link

Cross-Correlation performance #28

Closed damb closed 3 years ago

damb commented 3 years ago

This is a reminder for myself.

As @luca-s pointed out, there is still a way to improve the cross-correlation performance in order to allow the CPU to properly make use of vectorization:

Drop https://github.com/damb/scdetect/blob/03c8c444057083a4504b34bc2ef0b1b70ae79ba0/src/apps/scdetect/template.cpp#L335-L338
Properly initialize the cross-correlation loop

References:

https://github.com/swiss-seismological-service/scrtdd/blob/2fe488b558cb4d54452980bcb540d1c2801d0238/libs/hdd/waveform.cpp#L487-L642

damb commented 3 years ago

Note that this performance improvements are not relevant anymore since the cross-correlation is going to be implemented as an InplaceFilter. As a consequence, there won't be any branches within the cross-correlation loop, anyway.