Skip to content

fix(ops): Use correct C types for torch/tf Buffer - #18

Merged
papazof merged 1 commit into
mainfrom
fix_inference_buffer_objects
May 11, 2025
Merged

papazof merged 1 commit into
mainfrom
fix_inference_buffer_objects

Conversation

@papazof

@papazof papazof commented May 11, 2025

Copy link
Copy Markdown
Contributor

Use correct C types for Buffer classes of torch/tf ops

@papazof
papazof force-pushed the fix_inference_buffer_objects branch from 081898d to bc97ad0 Compare May 11, 2025 18:09
@papazof
papazof marked this pull request as ready for review May 11, 2025 18:11
@papazof
papazof requested a review from ananos May 11, 2025 18:11

@ananos ananos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @papazof

Use correct C types for `Buffer` classes of torch/tf ops

PR: #18
Signed-off-by: Kostis Papazafeiropoulos <papazof@gmail.com>
Reviewed-by: Anastassios Nanos <ananos@nubificus.co.uk>
Approved-by: Anastassios Nanos <ananos@nubificus.co.uk>
@github-actions
github-actions Bot force-pushed the fix_inference_buffer_objects branch from bc97ad0 to c3daaca Compare May 11, 2025 18:29
@github-actions

Copy link
Copy Markdown

Coverage report diff

Filename Stmts Miss Cover
ops/tf.py 0 -17 +8.30%
ops/torch.py 0 -17 +13.39%
TOTAL 0 -34 +2.83%

@papazof
papazof merged commit 7e1cb78 into main May 11, 2025
@papazof
papazof deleted the fix_inference_buffer_objects branch May 11, 2025 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants