Skip to content
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

Reduce work if only current season #47

Open
LasseEngboChr opened this issue Dec 18, 2024 · 0 comments
Open

Reduce work if only current season #47

LasseEngboChr opened this issue Dec 18, 2024 · 0 comments

Comments

@LasseEngboChr
Copy link
Contributor

In seasonal_onset.R there is an option to get results for the current season only. However, the analysis is first applied to all data and then the previous seasons are filtered out afterwards.

Suggestion: When results should only be returned for the current season then the input should be filtered at the top so that only the needed analysis is performed. One way could be to filter data to include the current season plus the k-1 last observations from the previous season.

LasseEngboChr pushed a commit that referenced this issue Dec 18, 2024
Focus on issues #47 to #50

### Checklist

* [ ] The PR passes all local unit tests
* [ ] I have documented any new features introduced
* [ ] If the PR adds a new feature, please add an entry in `NEWS.md`
* [ ] A reviewer is assigned to this PR
@LasseEngboChr LasseEngboChr mentioned this issue Dec 18, 2024
4 tasks
LasseEngboChr pushed a commit that referenced this issue Jan 6, 2025
Focus on issues #47 to #50

### Checklist

* [ ] The PR passes all local unit tests
* [ ] I have documented any new features introduced
* [ ] If the PR adds a new feature, please add an entry in `NEWS.md`
* [ ] A reviewer is assigned to this PR
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

No branches or pull requests

1 participant