Skip to content

Conversation

@nbroad1881
Copy link
Contributor

Needs to check string value.

@mryab mryab requested a review from artek0chumak May 6, 2025 21:26
@nbroad1881
Copy link
Contributor Author

nbroad1881 commented May 6, 2025

Oh I see the confusion. I wanted to pass a string, but it's designed to take one of [DownloadCheckpointType.DEFAULT, DownloadCheckpointType.MERGED, DownloadCheckpointType.ADAPTER]

I think string makes the most sense (and matches the cli), but having it accept either might be easiest for users.

@mryab mryab changed the title add .value Accept strings for checkpoint type on download May 7, 2025
@nbroad1881
Copy link
Contributor Author

@mryab , @artek0chumak ,

What do you think of it now?

Copy link
Member

@mryab mryab left a comment

Choose a reason for hiding this comment

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

@nbroad1881 Can you also bump a patch version in pyproject.toml, please?

@nbroad1881
Copy link
Contributor Author

@nbroad1881 Can you also bump a patch version in pyproject.toml, please?

1.5.7 -> 1.5.8

@mryab mryab merged commit 4eef896 into main May 13, 2025
10 checks passed
@mryab mryab deleted the nbroad/downloadcheckpointtype-value-fix branch May 13, 2025 18:33
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.

4 participants