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

Conversation

speediedan
Copy link
Contributor

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?

Minor fix for stale finetuning scheduler link, fix for finetuning scheduler tutorial tag capitalization and minor tutorial update referencing additional extension features.

Note that this PR should be merged in addition to the minor css enhancement PR to allow the submodule update PR to succeed.

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 🙃

@codecov
Copy link

codecov bot commented Jun 10, 2022

Codecov Report

Merging #176 (3acd8ec) into main (aa39ef1) will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##           main   #176   +/-   ##
===================================
  Coverage    75%    75%           
===================================
  Files         2      2           
  Lines       370    370           
===================================
  Hits        276    276           
  Misses       94     94           

references on nbval job, add reference to new fts release param
@speediedan speediedan force-pushed the fts_tutorial_update branch from 099f8ac to ff60373 Compare June 11, 2022 00:53
This was referenced Jun 11, 2022
@speediedan speediedan force-pushed the fts_tutorial_update branch from 90a218d to 7813019 Compare June 11, 2022 04:45
…ease due to bug in omegaconf 2.0.6 that requires omegaconfig >=2.1.0 which in turns requires hydra-core>=1.1.0 if it's installed (fts does not require hydra)
@speediedan speediedan force-pushed the fts_tutorial_update branch from 7813019 to a269bbb Compare June 11, 2022 23:35
Co-authored-by: Rohit Gupta <rohitgr1998@gmail.com>
@Borda Borda added the bug / fix Something isn't working label Jun 13, 2022
@Borda Borda enabled auto-merge (squash) June 13, 2022 11:58
@Borda Borda disabled auto-merge June 13, 2022 13:33
@Borda Borda enabled auto-merge (squash) June 13, 2022 14:58
@Borda Borda merged commit a2617fa into Lightning-AI:main Jun 13, 2022
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