Skip to content

Conversation

bigcat88
Copy link
Contributor

@bigcat88 bigcat88 commented Oct 4, 2025

Many nodes from this PR were tested. During conversion, minimal possible changes were made to avoid breaking such a large file.

Screenshot From 2025-10-04 19-43-50 Screenshot From 2025-10-04 19-53-31

Objects git diff:

object_info_difference.zip

@bigcat88 bigcat88 requested a review from Kosinkadink as a code owner October 4, 2025 16:54
@bigcat88
Copy link
Contributor Author

bigcat88 commented Oct 4, 2025

+label: Core

@comfy-pr-bot comfy-pr-bot added the Core Core team dependency label Oct 4, 2025
@Kosinkadink Kosinkadink changed the title [V3] convert nodes_customer_sampler.py to V3 schema [V3] convert nodes_custom_sampler.py to V3 schema Oct 4, 2025
@Kosinkadink
Copy link
Collaborator

Since this changes so many nodes + a lot of custom nodes affected at the same time, we should give 2 weeks heads up to custom nodes, and use that time for QA. We just need to make sure every node gets tested from this.

@bigcat88
Copy link
Contributor Author

bigcat88 commented Oct 5, 2025

We just need to make sure every node gets tested from this.

Should I test every node from this PR?

@Kosinkadink
Copy link
Collaborator

Yep, ideally every node should be tested.

@bigcat88 bigcat88 force-pushed the v3/nodes_custom_sampler branch from 23066aa to 2177439 Compare October 13, 2025 17:18
@bigcat88
Copy link
Contributor Author

bigcat88 commented Oct 13, 2025

Yep, ideally every node should be tested.

done, all nodes are good.
also added old_function=execution to each node, as in other PRs, to improve backwards compatibility.

This was referenced Oct 13, 2025
@Kosinkadink
Copy link
Collaborator

Node authors notified, will merge on Wednesday October 29th.

@Kosinkadink Kosinkadink added the Scheduled Merge PR is reviewed and ready, but will be merged at a specific time. label Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Core Core team dependency Scheduled Merge PR is reviewed and ready, but will be merged at a specific time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants