Skip to content
This repository was archived by the owner on Sep 1, 2023. It is now read-only.

Validate configure_sharded_model hook is overridden #12

Merged
merged 2 commits into from
Feb 27, 2023

Conversation

awaelchli
Copy link
Collaborator

Validates that the user has overridden the configure_sharded_model hook, as this is a requirement for the ColossalAI strategy: Lightning-AI/pytorch-lightning#16872

@awaelchli awaelchli requested a review from Borda as a code owner February 25, 2023 16:34
@awaelchli awaelchli added the enhancement New feature or request label Feb 25, 2023
@codecov
Copy link

codecov bot commented Feb 25, 2023

Codecov Report

Merging #12 (06df6bd) into main (fd7aa4f) will decrease coverage by 0%.
The diff coverage is 0%.

Additional details and impacted files
@@        Coverage Diff         @@
##           main   #12   +/-   ##
==================================
- Coverage    34%   33%   -0%     
==================================
  Files         4     4           
  Lines       298   300    +2     
==================================
  Hits        100   100           
- Misses      198   200    +2     

@Borda Borda merged commit 0552492 into main Feb 27, 2023
@Borda Borda deleted the feature/error-configure-sharded branch February 27, 2023 18:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants