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.

Sign up for GitHub

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

Jump to bottom

Update quarterstack pipeline for daily pipeline rev_08 changes #240

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

Update quarterstack pipeline for daily pipeline rev_08 changes #240

wants to merge 3 commits into from

Conversation

ljgray
Copy link
Contributor

@ljgray ljgray commented May 2, 2024

No description provided.

@ljgray ljgray force-pushed the ljg/quarterstack-updates branch 2 times, most recently from e9fc7cb to fb95547 Compare May 3, 2024 17:36
@ljgray ljgray marked this pull request as ready for review May 3, 2024 20:03
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch from bf1be05 to 429ad2e Compare May 4, 2024 18:31
@sjforeman
Copy link
Contributor

Suggested changes for updated rainfall flagging:

  1. Remove rain1mm_sd from flags in type: ch_pipeline.analysis.flagging.DataFlagger block
  2. Add FlagRainfall (feat(flagging): add pipeline task for rainfall flagging #244) after DataFlagger:
    - type: ch_pipeline.analysis.flagging.FlagRainfall
      in: sstream_mask3
      out: sstream_mask4
      params:
        accumulation_time: 30.0
        threshold: 1.0
  1. Updated later sstream_maskN references for consistency

@ljgray ljgray force-pushed the ljg/quarterstack-updates branch from 3b0aa25 to a9601c4 Compare May 13, 2024 19:37
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch from bb71a37 to c0325fb Compare July 3, 2024 18:53
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch from 34653a9 to d09045a Compare July 17, 2024 20:22
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch from d09045a to 3d948e5 Compare July 25, 2024 18:27
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch 3 times, most recently from 8f8427f to 7b68aa7 Compare August 7, 2024 18:57
@ljgray ljgray changed the title Update quarterstack config for daily pipeline rev_08 changes Update quarterstack pipeline for daily pipeline rev_08 changes Aug 9, 2024
@ljgray ljgray requested a review from sjforeman August 9, 2024 18:17
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch from 7b68aa7 to 86dc98d Compare August 13, 2024 20:51
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch 2 times, most recently from b336a17 to 0ab9da0 Compare August 26, 2024 22:18
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch from 0ab9da0 to a23a56b Compare September 18, 2024 18:19
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch from a23a56b to 8ec7150 Compare September 25, 2024 19:44
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch from 8ec7150 to 27d3b40 Compare September 27, 2024 22:38
@ljgray ljgray force-pushed the ljg/quarterstack-updates branch from 27d3b40 to 91b707c Compare October 7, 2024 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Arnab-half-blood-prince Arnab-half-blood-prince Arnab-half-blood-prince left review comments

@sjforeman sjforeman Awaiting requested review from sjforeman

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ljgray @sjforeman @Arnab-half-blood-prince