Skip to content

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

Closed
meggart wants to merge 3 commits into
mainfrom
fg/wlinreg
Closed

[Draft] Alternative rqa trend implementation respecting actual time stamps #73
meggart wants to merge 3 commits into
mainfrom
fg/wlinreg

Conversation

@meggart

@meggart meggart commented Feb 13, 2025

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
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
Copy Markdown
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.

@felixcremer

Copy link
Copy Markdown
Collaborator

I am closing this for now, because we decided to not go forward with it at the moment.

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