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

[New Feature]: Remove R3 DISP-S1 historical logic from batch_process lambda #928

Open
philipjyoon opened this issue Jul 22, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request needs triage Issue that requires triage

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

R3 DISP-S1 historical processing is now executed by a stand-alone CLI application as opposed to the batch_process lambda. batch_process lambda should now revert back to a state before the R3 logic was introduced:

  1. Cleaner, easier maintenance, etc
  2. The extra R3 code could trigger an edge-case bug in future. No reason to keep it around
  3. That R3 code doesn't work anyway because the historical processing logic has changed significantly after it was written
@philipjyoon philipjyoon added enhancement New feature or request needs triage Issue that requires triage labels Jul 22, 2024
@philipjyoon philipjyoon self-assigned this Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs triage Issue that requires triage
Projects
None yet
Development

No branches or pull requests

1 participant