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

[New Feature]: DISP-S1 re-triggering of previously submitted frames #948

Open
philipjyoon opened this issue Aug 15, 2024 · 2 comments
Open

[New Feature]: DISP-S1 re-triggering of previously submitted frames #948

philipjyoon opened this issue Aug 15, 2024 · 2 comments
Assignees
@philipjyoon
Labels
enhancement New feature or request needs triage Issue that requires triage pcm.r03-disp-s1

Comments

@philipjyoon
Copy link
Contributor

Checked for duplicates

Yes - I've already checked

Alternatives considered

Yes - and alternatives don't suffice

Related problems

No response

Describe the feature request

Rule #4: If we have a new granule for a batch that'd already been submitted,
submit again using the new granule only for the bursts that match, and only if the production time is later
@philipjyoon philipjyoon added enhancement New feature or request needs triage Issue that requires triage labels Aug 15, 2024
@philipjyoon philipjyoon self-assigned this Aug 15, 2024
philipjyoon added a commit that referenced this issue Aug 15, 2024
@philipjyoon
#948: Basic feature is done. Need to 1) create unit test 2) address c…
1bb9cfc 
…ase when some of the submitted granules are lost from GRQ ES
@philipjyoon
Copy link
Contributor Author

Also make sure to alter the hash of the re-triggered download job so that it's not deduped by hySDS

@philipjyoon
Copy link
Contributor Author

We discussed this during ASF-PST-ADT-SDS meeting a couple weeks ago and it wasn't decided whether this is even a desired behavior.

The conclusion was to:

  1. By default DO NOT retrigger
  2. But have option in software to toggle to that behavior (so probably another settings.yaml item)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philipjyoon philipjyoon

Labels
enhancement New feature or request needs triage Issue that requires triage pcm.r03-disp-s1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hhlee445 @philipjyoon