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

Fix BNAPost wrapper #97

Merged
merged 1 commit into from
Apr 8, 2024
Merged

Fix BNAPost wrapper #97

merged 1 commit into from
Apr 8, 2024

Conversation

rgreinho
Copy link
Member

@rgreinho rgreinho commented Apr 8, 2024

Fixes the BNAPost wrapper and adjust the logic to compute the scores
from the data instead of expecting them to be provided or set to 0.

Drive-by:

  • Adds a new step ResultPosted to the BrokenspokePipeline.

Signed-off-by: Rémy Greinhofer [email protected]

Fixes the BNAPost wrapper and adjust the logic to compute the scores
from the data instead of expecting them to be provided or set to 0.

Drive-by:
- Adds a new step `ResultPosted` to the BrokenspokePipeline.

Signed-off-by: Rémy Greinhofer <[email protected]>
@rgreinho rgreinho self-assigned this Apr 8, 2024
@pull-request-size pull-request-size bot added the size/L Large (100-499 lines of changes) label Apr 8, 2024
@rgreinho rgreinho merged commit 815616e into main Apr 8, 2024
10 checks passed
@rgreinho rgreinho deleted the fix-post-bna branch April 8, 2024 02:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Large (100-499 lines of changes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant