Skip to content
This repository was archived by the owner on Aug 28, 2025. It is now read-only.

Conversation

@ethanwharris
Copy link
Member

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

Attempting to fix hanging issue for publishing workflow

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@ethanwharris ethanwharris added the bug / fix Something isn't working label Sep 5, 2022
@codecov
Copy link

codecov bot commented Sep 5, 2022

Codecov Report

Merging #197 (499d793) into main (59d6427) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@         Coverage Diff         @@
##           main   #197   +/-   ##
===================================
  Coverage    75%    75%           
===================================
  Files         2      2           
  Lines       370    370           
===================================
  Hits        276    276           
  Misses       94     94           

@Borda
Copy link
Contributor

Borda commented Sep 5, 2022

Some timeout, probably make the training smaller?

@Borda
Copy link
Contributor

Borda commented Oct 25, 2022

let's update it with master, but strange that now it fails :/

@akihironitta akihironitta self-assigned this Oct 27, 2022
@akihironitta akihironitta enabled auto-merge (squash) October 27, 2022 17:20
@Borda
Copy link
Contributor

Borda commented Nov 4, 2022

It was carried with some more fixes in #208

@Borda Borda closed this Nov 4, 2022
auto-merge was automatically disabled November 4, 2022 22:56

Pull request was closed

@Borda Borda deleted the bugfix/flash_notebooks branch November 4, 2022 22:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug / fix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants