Skip to content

[QDP] PyTorch input format support #815

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
Open
400Ping wants to merge 13 commits into
base: main
Choose a base branch
from
Open

[QDP] PyTorch input format support #815

400Ping wants to merge 13 commits into from

Conversation

@400Ping
Copy link
Contributor

@400Ping 400Ping commented Jan 12, 2026 •
edited
Loading

Purpose of PR

As title

Related Issues or PRs

Related to #752

Changes Made

  • Bug fix
  • New feature
  • Refactoring
  • Documentation
  • Test
  • CI/CD pipeline
  • Other

Breaking Changes

  • Yes
  • No

Checklist

  • Added or updated unit tests for all changes
  • Added or updated documentation for all changes
  • Successfully built and ran all unit tests or manual tests locally
  • PR title follows "MAHOUT-XXX: Brief Description" format (if related to an issue)
  • Code follows ASF guidelines

@400Ping
PyTorch input format support
ce57ab1 
Signed-off-by: 400Ping <fourhundredping@gmail.com>
viiccwen
viiccwen suggested changes Jan 12, 2026
View reviewed changes
@400Ping
fix
961558a 
Signed-off-by: 400Ping <fourhundredping@gmail.com>
@400Ping
update github ci
352cf7c 
Signed-off-by: 400Ping <fourhundredping@gmail.com>
@400Ping
fix
0d698c7 
Signed-off-by: 400Ping <fourhundredping@gmail.com>
@400Ping 400Ping marked this pull request as draft January 12, 2026 19:42
@400Ping
fix
5c2ec52 
Signed-off-by: 400Ping <fourhundredping@gmail.com>
@400Ping 400Ping marked this pull request as ready for review January 12, 2026 19:46
@400Ping
Copy link
Contributor Author

400Ping commented Jan 12, 2026

locally tested, should pass CI by now

@400Ping
Copy link
Contributor Author

400Ping commented Jan 13, 2026

I included the ci fix in the pr, because the current ci doesn't have PyTorch related installed.

@400Ping
remove github workflow
b58cdf1 
Signed-off-by: 400Ping <fourhundredping@gmail.com>
@400Ping 400Ping mentioned this pull request Jan 13, 2026
Merged
14 tasks
@400Ping
fix conflict
76741e8 
Signed-off-by: 400Ping <fourhundredping@gmail.com>
@400Ping
Merge branch 'main' into qdp/refractor-pytorch-input
03c135d
@400Ping
Copy link
Contributor Author

400Ping commented Jan 13, 2026

locally tested, should pass CI by now

Need to add the CI check in this PR first #822

@400Ping
Copy link
Contributor Author

400Ping commented Jan 13, 2026

The current CI can't pass because it doesn't have libtorch.

CheyuWu
CheyuWu reviewed Jan 14, 2026
View reviewed changes
400Ping and others added 5 commits January 14, 2026 11:33
@400Ping
fix
fa50b28 
Signed-off-by: 400Ping <fourhundredping@gmail.com>
@400Ping
Merge branch 'qdp/refractor-pytorch-input' of github.com:400Ping/maho…
02c41f4 
…ut into qdp/refractor-pytorch-input
@400Ping
fix
4ed3346 
Signed-off-by: 400Ping <fourhundredping@gmail.com>
@400Ping
chore: trigger ci
c4c724c
@400Ping
Merge branch 'main' into qdp/refractor-pytorch-input
a16ed11
@guan404ming
Copy link
Member

guan404ming commented Jan 15, 2026

plz rebase and solve the precommit error, thanks!

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

Reviewers

@ryankert01 ryankert01 ryankert01 left review comments

+2 more reviewers

@CheyuWu CheyuWu CheyuWu left review comments

@viiccwen viiccwen viiccwen requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@400Ping @guan404ming @CheyuWu @ryankert01 @viiccwen