Skip to content

Conversation

@lstein
Copy link
Collaborator

@lstein lstein commented Oct 21, 2022

  • remove unsupported testtubelogger, use csvlogger instead
  • fix logic for parsing --gpus option so that it won't crash if trailing comma absent
  • change trainer accelerator from unsupported 'ddp' to 'auto'
  • closes Textual Inversion error (TestTubeLogger) #1168

- remove unsupported testtubelogger, use csvlogger instead
- fix logic for parsing --gpus option so that it won't crash if
  trailing comma absent
- change trainer accelerator from unsupported 'ddp' to 'auto'
Copy link
Contributor

@Any-Winter-4079 Any-Winter-4079 left a comment

Choose a reason for hiding this comment

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

LGTM

@Any-Winter-4079 Any-Winter-4079 merged commit fbea657 into development Oct 21, 2022
@Any-Winter-4079 Any-Winter-4079 deleted the fix-text-inversion-crashes branch October 21, 2022 14:35
Copy link

@wasyleque wasyleque left a comment

Choose a reason for hiding this comment

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

works fine, Thanks ! verified

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