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.

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

handle config.json failed to load #14525

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

w-e-w
Copy link
Collaborator

@w-e-w w-e-w commented Jan 3, 2024

Description

shows a error like this and rever default if config.json failed to load as opposed to crash
issue #14521

*** Could not load settings
*** The config file "B:\Stable Diffusion\Configs\config.json" is likely corrupted
*** It has been moved to the "tmp/config.json"
*** Reverting config to default
***
    Traceback (most recent call last):
      File "B:\GitHub\stable-diffusion-webui\modules\launch_utils.py", line 250, in list_extensions
        settings = json.load(file)
      File "C:\Programs\Python\3.10.6\lib\json\__init__.py", line 293, in load
        return loads(fp.read(),
      File "C:\Programs\Python\3.10.6\lib\json\__init__.py", line 346, in loads
        return _default_decoder.decode(s)
      File "C:\Programs\Python\3.10.6\lib\json\decoder.py", line 337, in decode
        obj, end = self.raw_decode(s, idx=_w(s, 0).end())
      File "C:\Programs\Python\3.10.6\lib\json\decoder.py", line 353, in raw_decode
        obj, end = self.scan_once(s, idx)
    json.decoder.JSONDecodeError: Expecting ',' delimiter: line 300 column 32 (char 10686)

---

@AUTOMATIC1111 AUTOMATIC1111 merged commit 1e7a8ce into dev Jan 4, 2024
6 checks passed
@AUTOMATIC1111 AUTOMATIC1111 deleted the handle-config.json-failed-to-load branch January 4, 2024 08:16
w-e-w added a commit that referenced this pull request Jan 26, 2024
@w-e-w w-e-w mentioned this pull request Jan 26, 2024
4 tasks
AUTOMATIC1111 added a commit that referenced this pull request Jan 26, 2024
@w-e-w w-e-w mentioned this pull request Feb 17, 2024
@pawel665j pawel665j mentioned this pull request Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants