Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix for train.controlnet.controlnet_v1_5_1node_100steps #9678

Merged
merged 6 commits into from
Jul 31, 2024

Conversation

rohitrango
Copy link
Contributor

What does this PR do ?

Fixes train.controlnet.controlnet_v1_5_1node_100steps CI test.

Collection: [Note which collection this PR will affect]

Changelog

  • Add specific line by line info of high level changes in this PR.

Usage

  • You can potentially add a usage example below
# Add a code snippet demonstrating how to use this 

GitHub Actions CI

The Jenkins CI system has been replaced by GitHub Actions self-hosted runners.

The GitHub Actions CI will run automatically when the "Run CICD" label is added to the PR.
To re-run CI remove and add the label again.
To run CI on an untrusted fork, a NeMo user with write access must first click "Approve and run".

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you add or update any necessary documentation?
  • Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)
    • Reviewer: Does the PR have correct import guards for all optional libraries?

PR Type:

  • New Feature
  • Bugfix
  • Documentation

If you haven't finished some of the above items you can still open "Draft" PR.

Who can review?

Anyone in the community is free to review the PR once the checks have passed.
Contributor guidelines contains specific people who can review PRs to various areas.

Additional Information

  • Related to # (issue)

@ericharper ericharper added 2.0.0rc1 bug Something isn't working labels Jul 10, 2024
@rohitrango rohitrango force-pushed the rohitrango/controlnet_cibug branch from 7dc6069 to 470034b Compare July 10, 2024 22:21
@rohitrango rohitrango changed the base branch from main to r2.0.0rc1 July 10, 2024 22:57
Copy link
Contributor

This PR is stale because it has been open for 14 days with no activity. Remove stale label or comment or update or this will be closed in 7 days.

@github-actions github-actions bot added the stale label Jul 25, 2024
Rohit Jena and others added 2 commits July 29, 2024 15:11
directly from the unet.

also added a checkpoint converter for SD that converts both SD and SDXL

Signed-off-by: Rohit Jena <[email protected]>
@rohitrango rohitrango force-pushed the rohitrango/controlnet_cibug branch from c01c3a3 to 95a8a98 Compare July 29, 2024 22:13
@rohitrango rohitrango changed the base branch from r2.0.0rc1 to main July 29, 2024 22:13
Victor49152
Victor49152 previously approved these changes Jul 29, 2024
@github-actions github-actions bot removed the stale label Jul 30, 2024
@Victor49152 Victor49152 merged commit e5b0fef into NVIDIA:main Jul 31, 2024
220 checks passed
xuanzic pushed a commit to xuanzic/NeMo that referenced this pull request Aug 1, 2024
* Fixed bug with controlnet where the model can be assumed to be saved
directly from the unet.

also added a checkpoint converter for SD that converts both SD and SDXL

Signed-off-by: Rohit Jena <[email protected]>

* delete unnecessary comments and messages

Signed-off-by: Rohit Jena <[email protected]>

* reinstate try-catch for loading missing modules

Signed-off-by: Rohit Jena <[email protected]>

* Apply isort and black reformatting

Signed-off-by: Victor49152 <[email protected]>

---------

Signed-off-by: Rohit Jena <[email protected]>
Signed-off-by: Victor49152 <[email protected]>
Co-authored-by: Rohit Jena <[email protected]>
Co-authored-by: Ming <[email protected]>
Co-authored-by: Victor49152 <[email protected]>
Signed-off-by: Vivian Chen <[email protected]>
monica-sekoyan pushed a commit that referenced this pull request Oct 14, 2024
* Fixed bug with controlnet where the model can be assumed to be saved
directly from the unet.

also added a checkpoint converter for SD that converts both SD and SDXL

Signed-off-by: Rohit Jena <[email protected]>

* delete unnecessary comments and messages

Signed-off-by: Rohit Jena <[email protected]>

* reinstate try-catch for loading missing modules

Signed-off-by: Rohit Jena <[email protected]>

* Apply isort and black reformatting

Signed-off-by: Victor49152 <[email protected]>

---------

Signed-off-by: Rohit Jena <[email protected]>
Signed-off-by: Victor49152 <[email protected]>
Co-authored-by: Rohit Jena <[email protected]>
Co-authored-by: Ming <[email protected]>
Co-authored-by: Victor49152 <[email protected]>
hainan-xv pushed a commit to hainan-xv/NeMo that referenced this pull request Nov 5, 2024
* Fixed bug with controlnet where the model can be assumed to be saved
directly from the unet.

also added a checkpoint converter for SD that converts both SD and SDXL

Signed-off-by: Rohit Jena <[email protected]>

* delete unnecessary comments and messages

Signed-off-by: Rohit Jena <[email protected]>

* reinstate try-catch for loading missing modules

Signed-off-by: Rohit Jena <[email protected]>

* Apply isort and black reformatting

Signed-off-by: Victor49152 <[email protected]>

---------

Signed-off-by: Rohit Jena <[email protected]>
Signed-off-by: Victor49152 <[email protected]>
Co-authored-by: Rohit Jena <[email protected]>
Co-authored-by: Ming <[email protected]>
Co-authored-by: Victor49152 <[email protected]>
Signed-off-by: Hainan Xu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Multi Modal Run CICD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants