Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Finn-Hecker
Copy link
Contributor

Summary

Fixed incorrect type hint syntax in MotionEncoder_tc.__init__() parameter list.

Changes

  • Line 647: Changed num_heads=int to num_heads: int
  • This corrects the parameter annotation from a default value assignment to proper type hint syntax

Details

The parameter was using assignment syntax (=) instead of type annotation syntax (:), which would incorrectly set the default value to the int class itself rather than annotating the expected type.

## Summary
Fixed incorrect type hint syntax in `MotionEncoder_tc.__init__()` parameter list.

## Changes
- Line 647: Changed `num_heads=int` to `num_heads: int` 
- This corrects the parameter annotation from a default value assignment to proper type hint syntax

## Details
The parameter was using assignment syntax (`=`) instead of type annotation syntax (`:`), which would incorrectly set the default value to the `int` class itself rather than annotating the expected type.
Copy link
Collaborator

@Kosinkadink Kosinkadink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice catch, merging. does not produce errors since the value is always passed in, but good for code health.

@Kosinkadink Kosinkadink merged commit 93d859c into comfyanonymous:master Oct 3, 2025
10 checks passed
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