Akegarasu / Akegarasu/lora-scripts

Suggestions - Regarding the default parameters of the WD Tagger

オープン
#643 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
enhancement
主要言語
Python
スター
6.1k
フォーク
699
PR マージ指標
30日以内にマージされた PR はありません

説明

I personally believe that two default parameters would benefit from adjustment, please consider it:

* **`escape_tag` :** Currently, this is enabled by default. **It would be better to set it to disabled by default.** Escaping parentheses during the tagging phase is unnecessary because the training process handles escaping automatically. If escaping is done during tagging, the entire process effectively escapes twice (as can be observed in the LoRA metadata).

* **`batch_output_action_on_conflict`:** Currently, this defaults to `copy`. This overwrites any existing tags with the inference results, and it has screwed my manually created tags twice (though, objectively, this was my mistake). **It would likely be better to default to `ignore`**, preserving existing tags, and allow users to manually select the `copy` mode when they intentionally want to overwrite them.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。