fix: num_image_with_embedding settings (it should be int as a config) - Githubissues

turingmotors / heron

Apache License 2.0

163 stars 25 forks source link

fix: num_image_with_embedding settings (it should be int as a config) #3

Closed Ino-Ichan closed 1 year ago

Ino-Ichan commented 1 year ago

fix: num_image_with_embedding settings (it should be int as a config)

Description

Purpose of the PR

I've modified the model to always take "num_image_withembedding" as an int.

Related Issue(s)

Nothing

Changes

Changed "num_image_with_embedding" in the config to accept an int.
Within the "git_llm" Config, if "num_image_with_embedding" is 1, it is set to None to ensure backward compatibility.

Model/Algorithm Performance

No model updates.

Dependencies

No dependencies update.

Reviewer Notes

None in particular.

Confirmed

[x] I have updated the documentation accordingly.
[x] I have adhered to the coding standards and guidelines of this project.
[x] I have added comments, especially in hard-to-understand areas.