Skip to content

Few-shot experiments: two issues (attn_mask in BiomedCLIPMultiHeadAttention, undefined VALIDATION in run_biomedical_training) #1

@KangST

Description

@KangST

Thank you for your excellent paper and for sharing the codebase. While running few-shot experiments, I encountered two issues that I wanted to report:

  1. TypeError in BiomedCLIPMultiHeadAttention.forward
TypeError: BiomedCLIPMultiHeadAttention.forward() got an unexpected keyword argument 'attn_mask'

As a temporary workaround, I added a no-op attn_mask parameter to the forward signature of BiomedCLIPMultiHeadAttention. Could you confirm whether this change is appropriate, or advise on a preferred fix?

  1. NameError for VALIDATION in run_biomedical_training
NameError: name 'VALIDATION' is not defined

Unlike run_training, run_biomedical_training does not define VALIDATION, which raises this error. Defining VALIDATION = False allowed the script to run without further issues. Is this the intended behavior, or should VALIDATION be set elsewhere?

Thank you again for your work and for any guidance you can share.

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