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.

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

Fix attribute error for detr #1008

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fix attribute error for detr #1008

wants to merge 1 commit into from

Conversation

meenakshiramanathan1
Copy link
Contributor

@meenakshiramanathan1 meenakshiramanathan1 commented Jan 6, 2025 •
edited
Loading

Model faces AttributeError: <class 'tvm.ir.op.Op'> has no attribute name_hint and zeros op is binded as global_var and passed as input params to tvmgen_default_forge_main as linked in the log below
detr.log

Fix for this is provided in this tvm PR, this PR fixes #1094

@meenakshiramanathan1 meenakshiramanathan1 changed the title Add tvm compiler flags for detr Fix attribute error for detr Jan 9, 2025
@meenakshiramanathan1 meenakshiramanathan1 force-pushed the mramanathan/detr branch 3 times, most recently from c53886b to b3bf31b Compare January 9, 2025 05:08
@meenakshiramanathan1 meenakshiramanathan1 marked this pull request as draft January 9, 2025 05:55
@meenakshiramanathan1 meenakshiramanathan1 marked this pull request as ready for review January 17, 2025 03:36
@meenakshiramanathan1 meenakshiramanathan1 force-pushed the mramanathan/detr branch 2 times, most recently from 937068d to 157306f Compare January 17, 2025 08:05
@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests823 ran490 passed333 skipped0 failed
TestResult
No test annotations available

@github-actions GitHub Actions
Copy link

TestsPassed ☑️Skipped ⚠️Failed ❌️
TT-Forge-FE Tests665 ran432 passed231 skipped2 failed
TestResult
TT-Forge-FE Tests
pytest
test_t5.test_t5_generation[t5-small]❌ failure
test_t5.test_t5_generation[google_flan_t5_small]❌ failure

@meenakshiramanathan1 meenakshiramanathan1 force-pushed the mramanathan/detr branch 3 times, most recently from ee276d2 to 554577e Compare January 20, 2025 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nvukobratTT nvukobratTT nvukobratTT left review comments

@pilkicTT pilkicTT Awaiting requested review from pilkicTT pilkicTT is a code owner

@dgolubovicTT dgolubovicTT Awaiting requested review from dgolubovicTT dgolubovicTT is a code owner

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

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Attribute error in detr
2 participants
@meenakshiramanathan1 @nvukobratTT