You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 6, 2021. It is now read-only.
Artifact Research notes being used while still snapshotted AP items are in inventory will mess up calculations.
Possible "solution" (which doesn't solve the problem): Show warning popup, which is reset when all items have been used. Actually solving this problem is tricky due to spellids being used and the API only returning the current values (with the latest Artifact Knowledge levels).
Better, but also tricky solution: Scan tooltips (different locales), using the Scanner algorithm, possibly both methods to find mismatches and use the scanned value when it's lower to detect snapshotted items?
Actual solution: Could cache all items as they are obtained and assign the current AK level to them. This will break if the user disables the addon (or it throws an error and is disabled as a result). It's also more difficult to implement and probably too elaborate for this rather obscure problem?
rdw-software
changed the title
Artifact Knowledge is wrongly applied to snapshotted items (see CF#4)
Artifact Knowledge is wrongly applied to snapshotted items - see CF#4
Aug 19, 2017
Artifact Research notes being used while still snapshotted AP items are in inventory will mess up calculations.
Possible "solution" (which doesn't solve the problem): Show warning popup, which is reset when all items have been used. Actually solving this problem is tricky due to spellids being used and the API only returning the current values (with the latest Artifact Knowledge levels).
Better, but also tricky solution: Scan tooltips (different locales), using the Scanner algorithm, possibly both methods to find mismatches and use the scanned value when it's lower to detect snapshotted items?
Actual solution: Could cache all items as they are obtained and assign the current AK level to them. This will break if the user disables the addon (or it throws an error and is disabled as a result). It's also more difficult to implement and probably too elaborate for this rather obscure problem?
See CF#4 for more details.
The text was updated successfully, but these errors were encountered: