-
Notifications
You must be signed in to change notification settings - Fork 8
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #34 from EtienneDosSantos/dev
Dev
- Loading branch information
Showing
7 changed files
with
40 additions
and
23 deletions.
There are no files selected for viewing
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,25 +1,34 @@ | ||
### Features to Add: | ||
|
||
**1. Image Metadata Storage** | ||
**3. Test Decoder Dtype Influence** ✔️ | ||
|
||
* **`torch.bfloat16` vs. `torch.float16`:** | ||
- [x] VRAM footprint | ||
- [x] Inference speed | ||
- [x] Image quality | ||
|
||
**2. Batch Size Fix (>1)** ✔️ | ||
|
||
* **Goal:** Restore the ability to generate multiple images per prompt. | ||
- [x] Not getting anywhere, opened [issue #7377](https://github.com/huggingface/diffusers/issues/7377) to hopefully get this resolved. | ||
- [x] **Issue Review:** Test provided solution to issue ([PR #7381](https://github.com/huggingface/diffusers/pull/7381))! Amazing work, thx [@DN6](https://github.com/DN6)! 🎉 | ||
* **Troubleshooting Steps:** | ||
- [ ] **Error Analysis:** Identify the specific error or unexpected behavior. | ||
- [ ] **Code Review:** Examine logic related to batch size handling. | ||
- [ ] **Dependency Check:** Ensure compatibility between any updated libraries and the batching functionality. | ||
|
||
**1. Image Metadata Storage** ✔️ | ||
|
||
* **Goal:** Embed essential generation parameters within generated images for reproducibility and analysis. | ||
* **Metadata to Include:** | ||
- [x] Seed | ||
- [x] Number of steps | ||
- [x] Model name | ||
- [x] CFG value | ||
- [ ] Sampler | ||
- [x] Sampler | ||
- [x] Prompt | ||
|
||
* **Implementation Steps:** | ||
- **Library Selection:** Research image metadata libraries (e.g., ExifWrite, PIL/Pillow). | ||
- **Integration:** Modify image generation code to write metadata. | ||
- **Testing:** Verify metadata is written and readable. | ||
|
||
**2. Batch Size Fix (>1)** | ||
|
||
* **Goal:** Restore the ability to generate multiple images per prompt. | ||
* **Troubleshooting Steps:** | ||
- [ ] **Error Analysis:** Identify the specific error or unexpected behavior. | ||
- [ ] **Code Review:** Examine logic related to batch size handling. | ||
- [ ] **Dependency Check:** Ensure compatibility between any updated libraries and the batching functionality. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters