Skip to content

Conversation

jongsoo-openai
Copy link
Collaborator

Follow-up #7795
Now transposed weight is supported, remove unnecessary assertion that mx weight should be col-major

New contributor declaration

  • I am not making a trivial change, such as fixing a typo in a comment.

  • I have written a PR description following these
    rules.

  • I have run pre-commit run --from-ref origin/main --to-ref HEAD.

  • Select one of the following.

    • I have added tests.
      • /test for lit tests
      • /unittest for C++ tests
      • /python/test for end-to-end tests
    • This PR does not need a test because FILL THIS IN.
  • Select one of the following.

    • I have not added any lit tests.
    • The lit tests I have added follow these best practices,
      including the "tests should be minimal" section. (Usually running Python code
      and using the instructions it generates is not minimal.)

Copy link
Contributor

@lezcano lezcano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a test?

@jongsoo-openai jongsoo-openai force-pushed the 0920_skip_col_maj_mx_w_check branch from 6346797 to 6778e93 Compare September 21, 2025 18:01
@jongsoo-openai jongsoo-openai changed the title [mxfp] remove col-major check for mx weight [mxfp] remove col-major assert for mx weight Sep 22, 2025
@jongsoo-openai jongsoo-openai marked this pull request as ready for review September 22, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants