• Chi's avatar
    Replace multiple variables with one variable. (#5715) · d72a24b7
    Chi authored
    
    
    * I added a new doc string to the class. This is more flexible to understanding other developers what are doing and where it's using.
    
    * Update src/diffusers/models/unet_2d_blocks.py
    
    This changes suggest by maintener.
    Co-authored-by: default avatarSayak Paul <spsayakpaul@gmail.com>
    
    * Update src/diffusers/models/unet_2d_blocks.py
    
    Add suggested text
    Co-authored-by: default avatarSayak Paul <spsayakpaul@gmail.com>
    
    * Update unet_2d_blocks.py
    
    I changed the Parameter to Args text.
    
    * Update unet_2d_blocks.py
    
    proper indentation set in this file.
    
    * Update unet_2d_blocks.py
    
    a little bit of change in the act_fun argument line.
    
    * I run the black command to reformat style in the code
    
    * Update unet_2d_blocks.py
    
    similar doc-string add to have in the original diffusion repository.
    
    * I enhanced the code by replacing multiple redundant variables with a single variable, as they all served the same purpose. Additionally, I utilized the get_activation function for improved flexibility in choosing activation functions.
    
    * Using as black package to reformated my file
    
    * reverte some changes
    
    * Remove conv_out_padding variables and using as conv_in_padding
    
    * conv_out_padding create and add them into the code.
    
    * run black command to solving styling problem
    
    * add little bit space between comment and import statement
    
    * I am utilizing the ruff library to address the style issues in my Makefile.
    
    ---------
    Co-authored-by: default avatarSayak Paul <spsayakpaul@gmail.com>
    Co-authored-by: default avatarYiYi Xu <yixu310@gmail.com>
    Co-authored-by: default avatarPatrick von Platen <patrick.v.platen@gmail.com>
    d72a24b7
unet_3d_condition.py 32.1 KB