fix(asset-creator): validate Trellis templates before uploading images - #58
Draft
zedarvates wants to merge 1 commit into
Draft
zedarvates wants to merge 1 commit into
zedarvates wants to merge 1 commit into
Conversation
|
This was referenced Sep 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Le pipeline Image → 3D envoyait l’image avant de charger sa recette Trellis2. Un fichier manquant ou mal formé pouvait donc provoquer un échec après transfert. Un preset inconnu sélectionnait aussi silencieusement
lowvram.Cette PR prépare le template avant tout appel client et avant la création du dossier de sortie. Elle refuse une image source absente, un preset inconnu, un JSON absent/mal formé, une structure d’éditeur incompatible ou un nœud d’image non modifiable. Le template préparé reste en mémoire ; après upload, son nom d’image est remplacé par celui renvoyé par le serveur, sans relire la recette.
Le README de l’addon corrige également l’affirmation que le workflow
trunk_onlyserait inclus. Les quatre recettes attendues ne figurent pas dans l’arbre examiné.Validation
5b6c83bd26f60f7eb5e4da53f958f2d3b06edb4a.552a39febeb8ec47e7cc28bf492cc4f39b008bae.74fb69688232d9d85b8ce361824d8e7cc9324afe.python -m unittest discover -s tests/asset_creator -v— 9 tests passent.git diff --cached --checket le lien documentaire local passent.Portée et commande des tests.
Suite du travail
Cette correction valide les prérequis locaux du chargeur existant. Celui-ci manipule encore
nodes/widgets_values: elle ne convertit pas un workflow d’éditeur au format API de/promptet ne prouve pas son exécution par ComfyUI.Il reste à retrouver les JSON réellement utilisés dans Asset Factory, identifier les versions de leurs nœuds/modèles, établir leur format de soumission puis fiabiliser les délais et la reprise des tâches. Aucun GLB, essai GPU, worker local ou raccord complet Asset Factory/MCP/CLI n’est validé ici.
La branche
developindiquée par CONTRIBUTING.md est absente des références distantes consultées ; le brouillon cible donc la branche existantemain.