Skip to content

[DRAFT] Slice operator tiling improvement #140

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
Draft
diaconuccalin wants to merge 29 commits into
base: devel
Choose a base branch
from
Draft

[DRAFT] Slice operator tiling improvement #140

diaconuccalin wants to merge 29 commits into from

Conversation

@diaconuccalin
Copy link
Contributor

@diaconuccalin diaconuccalin commented Dec 15, 2025

Describe the intent of your PR here.

Added

Changed

Fixed

PR Merge Checklist

  1. The PR is rebased on the latest devel commit and pointing to devel.
  2. Your PR reviewed and approved.
  3. All checks are passing.
  4. The CHANGELOG.md file has been updated.
  5. If the docker was modified, change back its link after review.

@diaconuccalin
Reordered some float kernel tests. Clean-up gitignore file
f9c7993
@diaconuccalin
Add various test cases for FP32 ReduceMean
cad3c2e
@diaconuccalin
Extended ReduceMean binder to support all-axis reduction (so missing …
5f185fb 
…axes parameter). Added ReduceMean tests. Other fixes
@diaconuccalin
ReduceMean parser fixes
029b598
@diaconuccalin
Implement reduction of singleton reduced sizes in ReduceMean. Warning…
41711b0 
… fix
@diaconuccalin
Added ReduceMean pass to remove if only singleton dimension reduction…
8a73d78 
… and related tests
@diaconuccalin
Improved ReduceMean template readability. Added ReduceMean test match…
d1b3e4d 
…ing the existent layer in TinyViT 5M
@diaconuccalin
Reverted changes from regular reduce mean template and moved them to …
a135cfa 
…float one
@diaconuccalin
Added parallelism to PULPOpen FP32 ReduceMean kernel
d359fc6
@diaconuccalin
FP ReduceMean PULPOpen template fix
4acb444
@diaconuccalin
Implement changes for tiling an uninitialized number of dimensions fo…
bb8ed1a 
…r ReduceMean PULPOpen FP operation
@diaconuccalin
Applied fixes to the PULPOpen FP32 ReduceMean template and tiling con…
60a41ac 
…straints. Added FP32 ReduceMean tests for Siracusa single buffer. Changed ReduceMean TinyViT test location. Decreased size of some ReduceMean tests
@diaconuccalin
Fixed test naming issues. Reduced dimensions of 2 ReduceMean tests
d93bf15
@diaconuccalin
Added CI tests for double buffering and L3 for PULPOpen FP32 ReduceMean
0e940d9
@diaconuccalin
Improved comments
cc379c2
@diaconuccalin
Removed more restrictive ReduceMean policy constraint
5185787
@diaconuccalin
Reintroduced more restrictive ReduceMean constraint
7a55995
@diaconuccalin
Updated changelog
849dc04
@diaconuccalin
Apply code rabbit suggestions
195ed2d
@diaconuccalin
Reorganized tests
a720a15
@diaconuccalin
Fixed test locations in text
7d0d72d
@diaconuccalin
Relocated softmax
d336243
@diaconuccalin
Quickfix
bb1ae8a
@diaconuccalin
Quickfix
d776090
@diaconuccalin
Quickfix
85b4796
@diaconuccalin
Fix formatting
1e0826e
@diaconuccalin diaconuccalin added this to the Release 0.2.1 milestone Dec 15, 2025
@diaconuccalin diaconuccalin self-assigned this Dec 15, 2025
@diaconuccalin diaconuccalin added Bug Something isn't working Feature Addition of new features labels Dec 15, 2025
@diaconuccalin diaconuccalin linked an issue Dec 15, 2025 that may be closed by this pull request
[BUG] Improper tiling implementation in PULPOpen for Slice and ReduceMean operators #134
Open
@diaconuccalin diaconuccalin changed the title Slice operator tiling improvement [DRAFT] Slice operator tiling improvement Dec 15, 2025
@Xeratec Xeratec moved this to In progress in Deeploy Dec 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Victor-Jung Victor-Jung Awaiting requested review from Victor-Jung Victor-Jung will be requested when the pull request is marked ready for review Victor-Jung is a code owner

@Xeratec Xeratec Awaiting requested review from Xeratec Xeratec will be requested when the pull request is marked ready for review Xeratec is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@diaconuccalin diaconuccalin

Labels

Bug Something isn't working Feature Addition of new features

Projects

Deeploy
Status: In progress

Milestone

Release 0.2.1

Development

Successfully merging this pull request may close these issues.

[BUG] Improper tiling implementation in PULPOpen for Slice and ReduceMean operators

1 participant

@diaconuccalin