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.

Sign up for GitHub

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

Jump to bottom

[HacktoberFest][Missing Docstrings] Autoencoder decode #5230

Closed
patrickvonplaten opened this issue Sep 29, 2023 · 1 comment · Fixed by #5242
Closed

[HacktoberFest][Missing Docstrings] Autoencoder decode #5230

patrickvonplaten opened this issue Sep 29, 2023 · 1 comment · Fixed by #5242
Assignees
@yiyixuxu @sayakpaul
Labels
hacktoberfest

Comments

@patrickvonplaten
Copy link
Contributor

patrickvonplaten commented Sep 29, 2023 •
edited
Loading

This feature request is included in this years Diffusers' Hacktoberfest 👨‍💻!

Hacktoberfest's purpose is to give upcoming open-source developers the opportunity to learn about open-source and programming by fixes simple issues (like this one).

How to solve the issue:

    1. Every issue has a description further below explaining the problem. First, you should try to understand the issue.
    1. Second, follow the contribution guide to learn how to contribute to Diffusers.\
    1. Claim this issue for you by opening a PR that links to this issue and writing "I am solving this issue in <link-to-pr>"

Issue

The decode method of AutoencoderKL is completely missing docstrings, can you add them?
freespirit pushed a commit to freespirit/diffusers that referenced this issue Sep 29, 2023
Add docstring for the AutoencoderKL's decode
f7ee6b0 
huggingface#5230
@freespirit freespirit mentioned this issue Sep 29, 2023
Merged
@freespirit
Copy link
Contributor

I am solving this issue in #5242

freespirit pushed a commit to freespirit/diffusers that referenced this issue Oct 2, 2023
Follow the style guidelines in AutoencoderKL's decode
7f2d5d3 
huggingface#5230
freespirit pushed a commit to freespirit/diffusers that referenced this issue Oct 2, 2023
Follow the style guidelines in AutoencoderKL's encode
6d3265b 
huggingface#5230
patrickvonplaten pushed a commit that referenced this issue Oct 2, 2023
@freespirit
Add docstring for the AutoencoderKL's decode (#5242)
37a787a 
* Add docstring for the AutoencoderKL's decode

#5230

* Follow the style guidelines in AutoencoderKL's decode

#5230

---------

Co-authored-by: stano <>
patrickvonplaten added a commit that referenced this issue Oct 2, 2023
@freespirit @patrickvonplaten
Add a docstring for the AutoencoderKL's encode (#5239)
7a4324c 
* Add docstring for the AutoencoderKL's encode

#5229

* Support Python 3.8 syntax in AutoencoderKL.decode type hints

Co-authored-by: Patrick von Platen <patrick.v.platen@gmail.com>

* Follow the style guidelines in AutoencoderKL's encode

#5230

---------

Co-authored-by: stano <>
Co-authored-by: Patrick von Platen <patrick.v.platen@gmail.com>
rchuzh99 pushed a commit to rchuzh99/diffusers that referenced this issue Oct 4, 2023
@freespirit
Add docstring for the AutoencoderKL's decode (huggingface#5242)
e73238f 
* Add docstring for the AutoencoderKL's decode

huggingface#5230

* Follow the style guidelines in AutoencoderKL's decode

huggingface#5230

---------

Co-authored-by: stano <>
rchuzh99 pushed a commit to rchuzh99/diffusers that referenced this issue Oct 4, 2023
@freespirit @patrickvonplaten
Add a docstring for the AutoencoderKL's encode (huggingface#5239)
f520a2c 
* Add docstring for the AutoencoderKL's encode

huggingface#5229

* Support Python 3.8 syntax in AutoencoderKL.decode type hints

Co-authored-by: Patrick von Platen <patrick.v.platen@gmail.com>

* Follow the style guidelines in AutoencoderKL's encode

huggingface#5230

---------

Co-authored-by: stano <>
Co-authored-by: Patrick von Platen <patrick.v.platen@gmail.com>
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this issue Dec 25, 2023
@freespirit
Add docstring for the AutoencoderKL's decode (huggingface#5242)
3465edd 
* Add docstring for the AutoencoderKL's decode

huggingface#5230

* Follow the style guidelines in AutoencoderKL's decode

huggingface#5230

---------

Co-authored-by: stano <>
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this issue Dec 25, 2023
@freespirit @patrickvonplaten
Add a docstring for the AutoencoderKL's encode (huggingface#5239)
55683af 
* Add docstring for the AutoencoderKL's encode

huggingface#5229

* Support Python 3.8 syntax in AutoencoderKL.decode type hints

Co-authored-by: Patrick von Platen <patrick.v.platen@gmail.com>

* Follow the style guidelines in AutoencoderKL's encode

huggingface#5230

---------

Co-authored-by: stano <>
Co-authored-by: Patrick von Platen <patrick.v.platen@gmail.com>
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this issue Apr 26, 2024
@freespirit
Add docstring for the AutoencoderKL's decode (huggingface#5242)
bce8402 
* Add docstring for the AutoencoderKL's decode

huggingface#5230

* Follow the style guidelines in AutoencoderKL's decode

huggingface#5230

---------

Co-authored-by: stano <>
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this issue Apr 26, 2024
@freespirit @patrickvonplaten
Add a docstring for the AutoencoderKL's encode (huggingface#5239)
a5f5c78 
* Add docstring for the AutoencoderKL's encode

huggingface#5229

* Support Python 3.8 syntax in AutoencoderKL.decode type hints

Co-authored-by: Patrick von Platen <patrick.v.platen@gmail.com>

* Follow the style guidelines in AutoencoderKL's encode

huggingface#5230

---------

Co-authored-by: stano <>
Co-authored-by: Patrick von Platen <patrick.v.platen@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yiyixuxu yiyixuxu

@sayakpaul sayakpaul

Labels
hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add docstring for the AutoencoderKL's decode freespirit/diffusers
4 participants
@freespirit @yiyixuxu @sayakpaul @patrickvonplaten