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] UNet2DConditionModel #5228

Closed
patrickvonplaten opened this issue Sep 29, 2023 · 2 comments · Fixed by #5248
Closed

[HacktoberFest][Missing Docstrings] UNet2DConditionModel #5228

patrickvonplaten opened this issue Sep 29, 2023 · 2 comments · Fixed by #5248
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 forward method of UNet2DConditionModel is missing a couple of docstrings, e.g. for class_lables, attention_mask ... -> can you add all the missing ones?
@patrickvonplaten patrickvonplaten changed the title [Missing Docstrings] UNet2DConditionModel [HacktoberFest][Missing Docstrings] UNet2DConditionModel Sep 29, 2023
@Nandika-A
Copy link
Contributor

Hello! I am working on this issue.

@a-r-r-o-w a-r-r-o-w mentioned this issue Sep 30, 2023
Merged
@a-r-r-o-w
Copy link
Member

I've solved this in #5248.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[HacktoberFest] Add missing docstrings to diffusers/models a-r-r-o-w/diffusers
3 participants
@patrickvonplaten @a-r-r-o-w @Nandika-A