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

Update litegraph to 0.7.26 #161

Merged
merged 3 commits into from
Jul 18, 2024
Merged

Update litegraph to 0.7.26 #161

merged 3 commits into from
Jul 18, 2024

Conversation

huchenlei
Copy link
Member

@huchenlei huchenlei commented Jul 18, 2024

Litegraph 0.7.26 fixes the node list serialization order issue. This PR adds a test for it. The test fails on 0.7.25 and passes on 0.7.26.

Copy link

socket-security bot commented Jul 18, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@comfyorg/litegraph@0.7.26 network 0 587 kB k1oride

🚮 Removed packages: npm/@comfyorg/litegraph@0.7.25

View full report↗︎

@huchenlei
Copy link
Member Author

Rendering of overlapped connection changed a bit. Example diff screenshot:
image

Common effect on litegraph side change. Probably need to fix later on the deterministic rendering.

@huchenlei huchenlei added the New Browser Test Expectations New browser test screenshot should be set by github action label Jul 18, 2024
@huchenlei huchenlei merged commit ab7436f into main Jul 18, 2024
2 checks passed
@huchenlei huchenlei deleted the lg branch July 18, 2024 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Browser Test Expectations New browser test screenshot should be set by github action
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant