Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Always raise from previous error #8751

Merged
merged 1 commit into from
Jul 1, 2024
Merged

Conversation

Wauplin
Copy link
Collaborator

@Wauplin Wauplin commented Jul 1, 2024

cc @sayakpaul it's always good practice to raise exceptions from the action exception. This should make the stacktrace easier to investigate. This PR does this in _get_model_file. It should help investigate why https://github.com/huggingface/diffusers/actions/runs/9731468292/job/26855960103?pr=8743#step:7:1780 failed.

@Wauplin Wauplin requested a review from sayakpaul July 1, 2024 08:31
@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@sayakpaul sayakpaul merged commit 4997975 into main Jul 1, 2024
17 of 18 checks passed
@Wauplin Wauplin deleted the always-raise-from-previous-error branch July 1, 2024 08:52
sayakpaul pushed a commit that referenced this pull request Dec 23, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants