Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Kernel mapping error resolve#42466

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.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
Aaraviitkgp wants to merge7 commits intohuggingface:main
base:main
Choose a base branch
Loading
fromAaraviitkgp:kernel_mapping_error_resolve

Conversation

@Aaraviitkgp
Copy link

@AaraviitkgpAaraviitkgp commentedNov 27, 2025
edited
Loading

issue:#42451

This PR fixes a bug in KernelConfig.create_compatible_mapping where defining a kernel mapping for multiple devices (e.g., both cuda and rocm) caused the last defined device to overwrite the active device configuration.

@Rocketknight1@MekkCyber

@AaraviitkgpAaraviitkgpforce-pushed thekernel_mapping_error_resolve branch from9623bfe to04e27cbCompareNovember 27, 2025 23:20
@AaraviitkgpAaraviitkgpforce-pushed thekernel_mapping_error_resolve branch from5a4a08a toe328d0cCompareNovember 27, 2025 23:25
@Aaraviitkgp
Copy link
Author

Aaraviitkgp commentedNov 27, 2025
edited
Loading

@Rocketknight1 I noticed that the run test issues that is occuring ColQwen2ForRetrieval: Tensor vlm.lm_head.weight seems to have nothing to do with the changes made in pr.

@Rocketknight1
Copy link
Member

cc@MekkCyber for kernels I think! also@Aaraviitkgp thatColQwen2 issue should be fixed now if you rebase, sorry about that!

Aaraviitkgp reacted with thumbs up emoji

@Aaraviitkgp
Copy link
Author

@Rocketknight1 the test_processor and test_tokenization have similar issue they fail but no changes in pr related to them can you look into that aswell 😁.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Aaraviitkgp@Rocketknight1

[8]ページ先頭

©2009-2025 Movatter.jp