Skip to content

Update checkpoint_utils.py to match new web urls in pretrained_models.py#2061

Open
hannadiamond wants to merge 5 commits into
Deci-AI:masterfrom
hannadiamond:patch-1
Open

Update checkpoint_utils.py to match new web urls in pretrained_models.py#2061
hannadiamond wants to merge 5 commits into
Deci-AI:masterfrom
hannadiamond:patch-1

Conversation

@hannadiamond

@hannadiamond hannadiamond commented Oct 23, 2024

Copy link
Copy Markdown
  • Update checkpoint_utils.py to match new web urls in pretrained_models.py
  • Fix unit tests that were trying to download from web urls leading no longer functioning s3 locations

BloodAxe
BloodAxe previously approved these changes Oct 24, 2024

@BloodAxe BloodAxe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, Thanks!

@hannadiamond
hannadiamond requested a review from BloodAxe October 25, 2024 00:18
@hannadiamond

Copy link
Copy Markdown
Author

@BloodAxe
Updated to fix unit tests

If you can take a look again and approve that would be great :)

@hdiamond-foundationstack

Copy link
Copy Markdown

@BloodAxe, @ofrimasad , @shaydeci

@AshkanEskandari AshkanEskandari left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

IndexError: list index out of range

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