Skip to content

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

New issue
New issue
Open
Bug
#137
#190
Open
[BUG] Improper tiling implementation in PULPOpen for Slice and ReduceMean operators#134
Bug
#137
#190
Assignees
diaconuccalin
Labels
BugSomething isn't working
Milestone
Release 0.2.2

Description

@diaconuccalin
diaconuccalin
opened on Nov 20, 2025

Describe the bug

The tiling system currently does not support list values as replacement variables, which affects the Slice and ReduceMean operators. These operators use arbitrary numbers of axes for input and output, which results in their data_in and data_out dimensions producing lists as replacement values.

To Reproduce

Run:

cd DeeployTest
python testRunner_tiled_siracusa.py -t Tests/testFloatDemoTinyViT/ --cores=8 --l1 4000

When checking the generated Network.c file, no tiling for Slice or ReduceMean operations

Expected behavior

Successful tiling for Slice and ReduceMean operations, that will be obtained by adjustings the templates and tiling constraints.

Additional context

Will need to also implement extra testing for Slice and ReduceMean operations.

Activity

  1. diaconuccalin
    self-assigned this
    on Nov 20, 2025
  2. diaconuccalin
    added
    BugSomething isn't working
    on Nov 20, 2025
  3. diaconuccalin
    added the
    Bug
    issue type on Nov 20, 2025
  4. diaconuccalin
    added a commit that references this issue on Nov 20, 2025

    Apply commit review and temporary fix of Slice and ReduceMean PULP co…

    fea1d26
  5. Xeratec
    added this to Deeployon Nov 21, 2025
  6. Xeratec
    added a commit that references this issue on Nov 21, 2025

    Demo TinyViT compatibility with tiled Siracusa (#124)

    e07cd13
  7. Xeratec
    linked a pull request that will close this issueFP32 ReduceMean operator improvement #137on Dec 12, 2025
  8. diaconuccalin
    linked a pull request that will close this issue[DRAFT] Slice operator tiling improvement #140on Dec 15, 2025
  9. Xeratec
    added a commit that references this issue on Dec 24, 2025

    FP32 ReduceMean operator improvement (#137)

    6db943f
  10. Xeratec
    closed this as completedin #137on Dec 24, 2025
  11. github-project-automation
    moved this to Done in Deeployon Dec 24, 2025
  12. Xeratec
    reopened this on Dec 24, 2025
  13. Xeratec
    moved this from Done to In progress in Deeployon Dec 24, 2025
  14. Xeratec
    added this to the Release 0.2.1 milestone on Dec 24, 2025

  15. Xeratec commented on Feb 3, 2026

    @Xeratec
    Xeratec
    on Feb 3, 2026
    Member

    @diaconuccalin If I remember correctly, this issue cannot yet be closed as you only implemented the tiling for the ReduceMean operator, correct?

  16. diaconuccalin commented on Feb 3, 2026

    @diaconuccalin
    diaconuccalin
    on Feb 3, 2026
    ContributorAuthor

    Exactly, once PR #140 is finished, we can mark this as solved. I did not have time to work on this recently, but I will try to get back to it this week or next week at the latest.

  17. Victor-Jung
    modified the milestones: Release 0.2.1, Release 0.2.2 on Feb 5, 2026
  18. diaconuccalin
    linked a pull request that will close this issueAdd 2D Autoencoder and GMM Integration on Siracusa Target #190on Nov 4, 2026

  19. diaconuccalin commented on Nov 4, 2026

    @diaconuccalin
    diaconuccalin
    on Nov 4, 2026
    ContributorAuthor

    Will be solved by PR #190

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

BugSomething isn't working

Type

Bug

Projects

  • Deeploy
    Status
    In progress

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions