Skip to content

had_12_paley is still used instead of the had_12 added in #5 #8

@swolchok

Description

@swolchok

Looks like the code generator just needs a one-line patch to replace had_12_paley by had_12, here: https://github.com/Dao-AILab/fast-hadamard-transform/blob/master/csrc/code_gen.py#L181

had_12 appears to match what QuaRot uses: https://github.com/spcl/QuaRot/blob/main/fake_quant/hadamard_utils.py#L152 (haven't exhaustively inspected it, but the first and second column match had_12 and not had_12_paley)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions