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

1.5.1 Update to nodes.py causes 'AttributeError: 'NoneType' object has no attribute 'get' for API queued workflows #377

Closed
Mattabyte opened this issue Feb 7, 2025 · 1 comment
Closed

1.5.1 Update to nodes.py causes 'AttributeError: 'NoneType' object has no attribute 'get' for API queued workflows #377

Mattabyte opened this issue Feb 7, 2025 · 1 comment

Comments

@Mattabyte
Copy link

When I queue a workflow via the API with a VHS node:

  "78": {
    "inputs": {
      "frame_rate": 24,
      "loop_count": 0,
      "filename_prefix": "vid",
      "format": "video/h264-mp4",
      "pix_fmt": "yuv420p",
      "crf": 20,
      "save_metadata": false,
      "trim_to_audio": false,
      "pingpong": false,
      "save_output": true,
      "images": [
        "73",
        0
      ]
    },
    "class_type": "VHS_VideoCombine",
    "_meta": {
      "title": "Video Combine 🎥🅥🅗🅢"
    }
  },

The node fails with

line 346: extra_options = extra_pnginfo.get('workflow', {}).get('extra', {})
AttributeError: 'NoneType' object has no attribute 'get'

This may be due to no default workflow or something.

Rolling back to 1.5.0 fixes this issue.
Mattabyte referenced this issue Feb 7, 2025
@AustinMroz
Prioritize builtin video_formats:
701f7e7 
Several other custom nodes have blindly copied VHS code and overwrite
the registered location for video formats. To solve this, the builtin
video_formats folder is checked separately and given priority over
folder_paths formats.

See #375

Bump version
AustinMroz added a commit that referenced this issue Feb 7, 2025
@AustinMroz
Fix options lookup when extra_pnginfo is None
571c17a 
Resolves #377
@AustinMroz
Copy link
Collaborator

Thanks

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

No branches or pull requests
2 participants
@AustinMroz @Mattabyte