Skip to content

Conversation

kohlisid
Copy link
Contributor

@kohlisid kohlisid commented Oct 22, 2024

Signed-off-by: Sidhant Kohli <sidhant.kohli@gmail.com>
Signed-off-by: Sidhant Kohli <sidhant.kohli@gmail.com>
Signed-off-by: Sidhant Kohli <sidhant.kohli@gmail.com>
Copy link

codecov bot commented Oct 22, 2024

Codecov Report

Attention: Patch coverage is 95.52239% with 3 lines in your changes missing coverage. Please review.

Project coverage is 94.24%. Comparing base (6ad59bd) to head (03e2cb8).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
pynumaflow/sourcetransformer/servicer/_servicer.py 95.38% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #196      +/-   ##
==========================================
- Coverage   94.28%   94.24%   -0.04%     
==========================================
  Files          56       56              
  Lines        2222     2259      +37     
  Branches      141      145       +4     
==========================================
+ Hits         2095     2129      +34     
- Misses         93       96       +3     
  Partials       34       34              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Sidhant Kohli <sidhant.kohli@gmail.com>
Signed-off-by: Sidhant Kohli <sidhant.kohli@gmail.com>
@kohlisid kohlisid changed the title [WIP] chore: streaming transformer chore: streaming transformer Oct 22, 2024
Signed-off-by: Sidhant Kohli <sidhant.kohli@gmail.com>
@kohlisid kohlisid marked this pull request as ready for review October 22, 2024 04:47
@kohlisid kohlisid requested a review from KeranYang October 22, 2024 04:55
Signed-off-by: Sidhant Kohli <sidhant.kohli@gmail.com>
@kohlisid kohlisid requested a review from ab93 October 22, 2024 23:36
@vigith vigith changed the title chore: streaming transformer feat: streaming transformer Oct 23, 2024
@vigith vigith merged commit a1d3eb5 into numaproj:main Oct 23, 2024
11 checks passed
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

Successfully merging this pull request may close these issues.

3 participants