Issue

For some reason the markers used within the inventory UI are sometimes cleared early. Not sure if this is caused by the game itself or if parts of the code are not in order.

Possible Solution

This requires some more testing and debugging. One has to determine what actually causes the markers to be reset. One attempt is to prevent clearing the array when the marking is still done, which also delays the opening of the menu.

Proposal

Currently the plugin operates without any kind of configuration, but with the proposed feature in issue #4 a configuration is necessary. Additional features could include toggling the disabling of the vanilla function, if only armors/weapons should be compared, and maybe some options relating to sorting and/or performance.

Proposal

The plugin so far only compares the different weapon and armor types, but leaves out bows, crossbows, and ammunition for these weapons. It would be reasonable to add those in before the release.

Issue

The function LogMessage() is not working as intended. While the timestamps work, the combination of the input and the timestamp is not. Whatever is handed into _Message() seems to be garbled mess.

Possible Solution

Implement an actually decent way of combining the input parameters with the timestamp.

Proposal

Currently the plugin compares the base values of each item, but this means that any improvements through like tempering or similar are not taken into account. An alternative comparison should be sought out to allow for users to select their preferred compare.

Issue

Armors can occupy multiple different slots at once, so the number used for the Switch Cases are not always correct and thus the armor piece is not recognized. The issue can be seen within the plugin's logs:

Current Armor "Amulet of Dibella (Flawless)" uses slot(s) 32
Current Armor "Backpack, Brown Fur (Exquisite)" uses slot(s) 131072
Current Armor "Dragon Hunter Armor (Flawless)" uses slot(s) 4
Current Armor "Einherjar Brigandine Dark Gauntlets (Flawless)" uses slot(s) 8
Current Armor "Einherjar Brigandine Dark Hood (Flawless)" uses slot(s) 2050
Current Armor "Forester Aketon Brown (Fine)" uses slot(s) 4194308
Current Armor "Forester Mantle Green" uses slot(s) 2
Current Armor "Forester Solid Boots Black" uses slot(s) 128
Current Armor "Forester Wristguards (Superior)" uses slot(s) 8
Current Armor "Gauldur Amulet Fragment" uses slot(s) 32
Current Armor "Goblet of Infinite Grace" uses slot(s) 0
Current Armor "Gold Emerald Ring" uses slot(s) 64
Current Armor "Leader of the Pack (Exquisite)" uses slot(s) 64
Current Armor "Necklace of Health" uses slot(s) 32
Current Armor "Ring of Dexterity (Flawless)" uses slot(s) 64
Current Armor "Ring of Extreme Wielding (Superior)" uses slot(s) 64
Current Armor "Saarthal Amulet" uses slot(s) 32
Current Armor "Scaled Boots (Epic)" uses slot(s) 128
Current Armor "Travel Lantern" uses slot(s) 0
Current Armor "Vvardenfell Glass Gauntlets" uses slot(s) 8

All slots are assigned a number in the form of n². If multiple slots are used, the numbers are added together.

Possible Solutions

There are three ways I see:

1. Hardcode all practical combinations. Awful idea.
2. Use arithmetic to determine which slots are used within an armor piece. Might not be the best approach and could pose issues with the Switch Case.
3. Find another field to use as the determining factor. The in-game (Sky)UI shows a more easily identifiable type to allow sorting of the armor. This seems like the best option.

Proposal

While the player inventory is already working, the menus for containers and trading are still not implemented. Implementing them requires some reverse engineering as it seems that these have not been fully defined by the SKSE team or other developers. This could take more time than wished, might consider releasing an inventory only version.

Issue

Since the last commit 6cfc13d the game crashes whenever the plugin is active and one opens the inventory menu. Logs seem to indicate that this issue arises within the weapon comparison, disabling this part of the plugin fixes the crashes. What exactly is causing the issues is not known yet.

Proposal

Some mods and overhauls seem to expand upon the various weapon and armor types already found within the game. Examples include Cloaks of Skyrim, Winter is Coming, Perkus Maximus, and others. Perkus Maximus splits bows into Shortbows and Longbows, but how each is defined and if it is possible to include these efficiently or as option is left to be seen after main release.