Skip to content

[Draft] Alternative rqa trend implementation respecting actual time stamps #73

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

meggart
Copy link
Collaborator

@meggart meggart commented Feb 13, 2025

Just to collect the results of our experiments with @felixcremer. This is mostly for documentation purposes and does not necessarily need to be merged since the resulting algorithm is 20x slower than the current version.

@danlooo danlooo changed the title Alternative rqa trend implementation respecting actual time stamps [Draft] Alternative rqa trend implementation respecting actual time stamps Feb 13, 2025
@felixcremer
Copy link
Collaborator

As mentioned at today's meeting. It might be worthwhile to test the tile wide performance with this branch and teat it against the faster implementation

@felixcremer felixcremer linked an issue Feb 14, 2025 that may be closed by this pull request
@felixcremer
Copy link
Collaborator

I am wondering whether we could improve the speed of this implementation by precomputing the x mean and x squared mean, because this should be the same for all pixels.
From what I understood is the main difference between these algorithm, that with the time series aware version we have to actually compute the x mean and with the standard version that is much easier to compute.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

What should be the default indir for EODC
2 participants