Skip to content

LDM Bert config.json path #675

Closed
Closed
LDM Bert config.json path#675
Assignees
patil-suraj
Labels
bugSomething isn't working

Description

@ryanrussell
ryanrussell
opened on Sep 29, 2022

Describe the bug

Problem

There is a reference to an LDM Bert that 404's

src/diffusers/pipelines/latent_diffusion/pipeline_latent_diffusion.py:    "ldm-bert": "https://huggingface.co/ldm-bert/resolve/main/config.json",

I was able to locate a config.json at https://huggingface.co/valhalla/ldm-bert/blob/main/config.json

Is this the correct config.json?

Notes for reviewer

Happy to send a PR if needed to update, feel free to do on your own if it's faster/easier :)

Reproduction

na

Logs

na

System Info

na

Activity

  1. ryanrussell
    added
    bugSomething isn't working
    on Sep 29, 2022

  2. patrickvonplaten commented on Sep 29, 2022

    @patrickvonplaten
    patrickvonplaten
    on Sep 29, 2022
    Contributor

    Yes this seems wrong, let's update it (cc @patil-suraj )

  3. patrickvonplaten
    assigned
    patil-suraj
    on Sep 29, 2022
  4. patil-suraj
    closed this as completedin 877bec8on Sep 30, 2022
  5. prathikr
    added a commit that references this issue on Oct 26, 2022

    refactor: update ldm-bert `config.json` url closes huggingface#675 (h…

    23514cf
  6. yoonseokjin
    added a commit that references this issue on Dec 25, 2023

    refactor: update ldm-bert `config.json` url closes huggingface#675 (h…

    a0e4da6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions