Skip to content

fix: PrintExpData node crashes with AttributeError on ExpressionSet#106

Open
deepme987 wants to merge 1 commit intoPowerHouseMan:mainfrom
deepme987:fix/exp-attribute-typo
Open

fix: PrintExpData node crashes with AttributeError on ExpressionSet#106
deepme987 wants to merge 1 commit intoPowerHouseMan:mainfrom
deepme987:fix/exp-attribute-typo

Conversation

@deepme987
Copy link
Copy Markdown

Bug

The PrintExpData node is completely broken — it crashes immediately with:

AttributeError: 'ExpressionSet' object has no attribute 'exp'

Steps to Reproduce

  1. Add an ExpData node with any non-zero values
  2. Connect it to a PrintExpData: node
  3. Execute the workflow
  4. Node crashes every time

Root Cause

In nodes.py line 634, PrintExpData.run() references exp.exp:

e = exp.exp * 1000

But the ExpressionSet class (line 475) defines its attributes as .e, .r, .s, .t — there is no .exp attribute:

class ExpressionSet:
    def __init__(self, erst=None, es=None):
        if es != None:
            self.e = copy.deepcopy(es.e)
            self.r = copy.deepcopy(es.r)
            self.s = copy.deepcopy(es.s)
            self.t = copy.deepcopy(es.t)

All other nodes (ExpData, ExpressionEditor, AdvancedLivePortrait) correctly use .e. This is a typo in PrintExpData only.

Fix

One-line change: exp.expexp.e

Tested

Verified the fix locally on ComfyUI 0.14.1 with Python 3.12, PyTorch 2.10+cu130, RTX 4090. The PrintExpData: node now correctly prints expression data without crashing.

Environment

  • ComfyUI Version: 0.14.1
  • OS: Windows 11
  • Python: 3.12.11
  • PyTorch: 2.10.0+cu130

The `PrintExpData` node crashes with `AttributeError: 'ExpressionSet'
object has no attribute 'exp'` because it references `exp.exp` instead
of the correct attribute `exp.e`.

The `ExpressionSet` class defines its expression tensor as `.e` (along
with `.r`, `.s`, `.t`), and all other nodes (`ExpData`, `ExpressionEditor`,
etc.) correctly use `.e`. This was a typo in `PrintExpData.run()` only.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant