niftools / nifskope

A git repository for nifskope.
http://www.niftools.org
Other
546 stars 232 forks source link

Unsupported NiProperty based blocks #78

Open neomonkeus opened 9 years ago

neomonkeus commented 9 years ago
hexabits commented 9 years ago

@ttl269 Any input on priority for implementing these? A "Most used" property list perchance? I will do an ASCII search in my NIF folders for these properties but if I can't find all of them in any actual NIF files I may not be able to safely implement them.

They will largely be placeholder classes however. No actual functionality introduced for them.

As I find examples of NIF files which use them I will try to update the list above with example NIFs for testing.

ttl269 commented 9 years ago
Game Property Number of occurences
Skyrim BSWaterShaderProperty 33
Skyrim BSSkyShaderProperty 4
Fallout3 WaterShaderProperty 26
Fallout3 TallGrassShaderProperty 9
Fallout3 TileShaderProperty 5
Fallout3 SkyShaderProperty 3
Fallout3 NiFogProperty 1
Fallout3 Lighting30ShaderProperty 2
Oblivion NiDitherProperty 15

searched in: Skyrim including all DLCs, Fallout3 only base game (no DLC, no FNV), Oblivion including Shivering Isles

@jonwd7 I think that highest priority should have both Skyrim related properties BSWaterShaderProperty (you have implemented it yet in experimental version of Nifskope) and BSSkyShaderProperty, because Skyrim is most recent game. Only avoiding Nifskope's warning message would be sufficient for now.

hexabits commented 6 years ago

Just commenting that both #124 and #100 are essentially duplicates of this. They are not errors, but end-users don't understand this, so in dev7 I have simply disabled the warning in release builds. The warning comes from the renderer not knowing what to do with the property.

JayFoxRox commented 5 years ago

Futurama also uses NiShadeProperty