egocarib
commented
2 months ago Looked into this a bit more and going to divide up the current "Reeds" category as follows:
- "Reeds" category will be associated only with cattails and papyrus
- A new "Tule" category will be added, since tule has other interesting uses (namely, thatch)
- Sedge will no longer be included in any category, since it's basically the same as grass and there's no good reason to track it
Sedge is currently captured by the "Reeds" marker category (along with cattails, papyrus, and tule), but it probably doesn't make sense to be in that category because it drops only dry grass.
This happens because it is part of the
tallplant-*json family, but it should probably be excluded from the mod's marker logic.https://github.com/egocarib/Vintage-Story-Mods/blob/fa54cf088465a63b729e32d9989a011f7cb6f574/mods/automapmarkers/src/Utilities/ThingIdentifier.cs#L391