Conversation
- 通常GT,titan replica3,icarus_4thannivにLoreの条件を追加
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! このプルリクエストは、OptiFineのカスタムアイテムテクスチャ（CIT）のマッチングロジックを改善することを目的としています。特定のアイテムにLore（説明文）の条件を追加することで、テクスチャがより正確に適用されるようになり、アイテムの識別精度が向上します。これにより、ユーザーは意図したカスタムテクスチャを確実に表示できるようになります。 Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces version v18.31 in the update log, focusing on improving texture assignment match patterns. The changes involve adding or modifying nbt.display.Lore properties in various item configuration files (armors, tools, weapons) to use iregex patterns for matching item descriptions. The review comments suggest that the iregex patterns might not be robust enough when Minecraft color codes are present in the Lore text. It is recommended to use ipattern for more reliable matching, especially for fixed strings, and to consider splitting properties files for complex OR logic patterns, as ipattern does not support it.
| texture.diamond_layer_2=diamond_layer_2 | ||
| nbt.display.Name=APOLLO | ||
| weight=-1 No newline at end of file | ||
| nbt.display.Lore.1=iregex:(装備中は満腹度自動回復と|右クリックで1個取り出し) |
There was a problem hiding this comment.
iregexパターンは、LoreのテキストにMinecraftのカラーコードが含まれているとマッチしません。カラーコードが存在する可能性がある場合は、ipatternの使用を検討してください。ipatternはORロジック（|）をサポートしていないため、堅牢なマッチングを保証するには、このプロパティファイルを2つに分割し、それぞれのLore条件に1つずつ対応させる必要があるかもしれません。例えば、apollo_lore1.propertiesとapollo_lore2.propertiesのようにです。
| items=minecraft:diamond_helmet | ||
| texture=diamond_helmet | ||
| nbt.display.Name=APOLLO | ||
| nbt.display.Lore.1=iregex:(装備中は満腹度自動回復と|右クリックで1個取り出し) |
There was a problem hiding this comment.
iregexパターンは、LoreのテキストにMinecraftのカラーコードが含まれているとマッチしません。カラーコードが存在する可能性がある場合は、ipatternの使用を検討してください。ipatternはORロジック（|）をサポートしていないため、堅牢なマッチングを保証するには、このプロパティファイルを2つに分割し、それぞれのLore条件に1つずつ対応させる必要があるかもしれません。例えば、apollo_lore1.propertiesとapollo_lore2.propertiesのようにです。
| texture.diamond_layer_2=diamond_layer_2 | ||
| nbt.display.Name=HERCULES | ||
| weight=-1 No newline at end of file | ||
| nbt.display.Lore.1=iregex:(装備中は跳躍力が付与されます|右クリックで1個取り出し) |
There was a problem hiding this comment.
iregexパターンは、LoreのテキストにMinecraftのカラーコードが含まれているとマッチしません。カラーコードが存在する可能性がある場合は、ipatternの使用を検討してください。ipatternはORロジック（|）をサポートしていないため、堅牢なマッチングを保証するには、このプロパティファイルを2つに分割し、それぞれのLore条件に1つずつ対応させる必要があるかもしれません。例えば、hercules_lore1.propertiesとhercules_lore2.propertiesのようにです。
| items=minecraft:diamond_boots | ||
| texture=diamond_boots | ||
| nbt.display.Name=HERCULES | ||
| nbt.display.Lore.1=iregex:(装備中は跳躍力が付与されます|右クリックで1個取り出し) |
There was a problem hiding this comment.
iregexパターンは、LoreのテキストにMinecraftのカラーコードが含まれているとマッチしません。カラーコードが存在する可能性がある場合は、ipatternの使用を検討してください。ipatternはORロジック（|）をサポートしていないため、堅牢なマッチングを保証するには、このプロパティファイルを2つに分割し、それぞれのLore条件に1つずつ対応させる必要があるかもしれません。例えば、hercules_lore1.propertiesとhercules_lore2.propertiesのようにです。
| texture=armor | ||
| nbt.display.Name=ICARUS | ||
| weight=-1 No newline at end of file | ||
| nbt.display.Lore.1=iregex:(装備中は火炎耐性が付与されます|右クリックで1個取り出し) |
There was a problem hiding this comment.
iregexパターンは、LoreのテキストにMinecraftのカラーコードが含まれているとマッチしません。カラーコードが存在する可能性がある場合は、ipatternの使用を検討してください。ipatternはORロジック（|）をサポートしていないため、堅牢なマッチングを保証するには、このプロパティファイルを2つに分割し、それぞれのLore条件に1つずつ対応させる必要があるかもしれません。例えば、icarus_lore1.propertiesとicarus_lore2.propertiesのようにです。
| items=minecraft:diamond_pickaxe | ||
| texture=thanatos | ||
| nbt.display.Name=THANATOS | ||
| nbt.display.Lore.1=iregex:(最大体力\\(中\\)増加|右クリックで1個取り出し) |
There was a problem hiding this comment.
iregexパターンは、LoreのテキストにMinecraftのカラーコードが含まれているとマッチしません。カラーコードが存在する可能性がある場合は、ipatternの使用を検討してください。ipatternはORロジック（|）をサポートしていないため、堅牢なマッチングを保証するには、このプロパティファイルを2つに分割し、それぞれのLore条件に1つずつ対応させる必要があるかもしれません。例えば、thanatos_lore1.propertiesとthanatos_lore2.propertiesのようにです。
| items=minecraft\:diamond_axe | ||
| texture=titan | ||
| nbt.display.Name=TITAN | ||
| nbt.display.Lore.1=iregex:(敵を空中へ打ち上げます|右クリックで1個取り出し) |
There was a problem hiding this comment.
iregexパターンは、LoreのテキストにMinecraftのカラーコードが含まれているとマッチしません。カラーコードが存在する可能性がある場合は、ipatternの使用を検討してください。ipatternはORロジック（|）をサポートしていないため、堅牢なマッチングを保証するには、このプロパティファイルを2つに分割し、それぞれのLore条件に1つずつ対応させる必要があるかもしれません。例えば、titan_lore1.propertiesとtitan_lore2.propertiesのようにです。
| nbt.display.Lore.*=耐久無限 | ||
| nbt.display.Lore.*=iregex:.*クリスマスイベント特別限定品 |
There was a problem hiding this comment.
カラーコードに対応するために、マッチングをより堅牢にするにはipatternの使用を推奨します。iregexはMinecraftのカラーコードを処理せず、プレーンな文字列照合はフォーマットが存在すると失敗します。両方の条件にipatternを使用することで、マッチングの信頼性が高まります。
nbt.display.Lore.*=ipattern:*耐久無限*
nbt.display.Lore.*=ipattern:*クリスマスイベント特別限定品*
| texture.bow_pulling_1=artemis_pulling_1 | ||
| texture.bow_pulling_2=artemis_pulling_2 | ||
| nbt.display.Name=ARTEMIS | ||
| nbt.display.Lore.1=iregex:(打つと花火があがります|右クリックで1個取り出し) |
There was a problem hiding this comment.
iregexパターンは、LoreのテキストにMinecraftのカラーコードが含まれているとマッチしません。カラーコードが存在する可能性がある場合は、ipatternの使用を検討してください。ipatternはORロジック（|）をサポートしていないため、堅牢なマッチングを保証するには、このプロパティファイルを2つに分割し、それぞれのLore条件に1つずつ対応させる必要があるかもしれません。例えば、artemis_lore1.propertiesとartemis_lore2.propertiesのようにです。
| items=minecraft:diamond_sword | ||
| texture=pluto | ||
| nbt.display.Name=PLUTO | ||
| nbt.display.Lore.1=iregex:(攻撃時に敵をウィザー状態にします|右クリックで1個取り出し) |
There was a problem hiding this comment.
iregexパターンは、LoreのテキストにMinecraftのカラーコードが含まれているとマッチしません。カラーコードが存在する可能性がある場合は、ipatternの使用を検討してください。ipatternはORロジック（|）をサポートしていないため、堅牢なマッチングを保証するには、このプロパティファイルを2つに分割し、それぞれのLore条件に1つずつ対応させる必要があるかもしれません。例えば、pluto_lore1.propertiesとpluto_lore2.propertiesのようにです。
|
Warning Gemini encountered an error creating the review. You can try again by commenting |
1 participant
このPRの変更点と理由:
通常GT，titan replica3，icarus_4thanniv にLoreの条件を追加し，マッチパターンの改善を行った