SteveTownsend / SmartHarvestSE

AutoHarvest 2 Skyrim SE mod ported to CommonLibSSE
GNU General Public License v3.0
18 stars 7 forks source link

Nothing glowing. #456

Closed Raveness69 closed 8 months ago

Raveness69 commented 9 months ago

Everything else is working great, but no glow at all. Below is the trace log from the logging mod.

SmartHarvestSE.log

SteveTownsend commented 9 months ago

Seems like not everybody is having this problem. Could be timing of TaskInterface is not the same as script invocation? Check why full diagnostics are not coming out from Logging DLL. Try to make the logic more resilient - don't discard the queued glow requests, handle dupes that arrive if stuff is already queued.

SteveTownsend commented 8 months ago

Here's output after changing the log-level setup to just alter my log. Maybe this will fix the missing debug logs?

image

SteveTownsend commented 8 months ago

Improved diagnostics in this area for v5.6.0. Not clear to me what is going on.

SteveTownsend commented 8 months ago

Could be related to script interactions analogous to #417.

SteveTownsend commented 8 months ago

Leaving this open for next milestone - may be able to close, depending on feedback on v5.6.0

PoptartJuniper commented 8 months ago

I'm linking my logs from my drive, they are way too large for github apparently. The last version I actually played with was 5.1 and the glow works. Switching back to 5.1 activates the glow again.

https://drive.google.com/drive/folders/1aDYS9sBKVhkNeSi8f3TFWtbjdP8VoIfN?usp=sharing

SteveTownsend commented 8 months ago

Thanks for the log, I should be able to hotfix this later on today by relaxing the check on whether it's OK to glow items.

2024-03-24 12:30:00.402    trace  52596 IsActivateControlsEnabled false
2024-03-24 12:30:00.402  warning  52596 Delay 31 queued Glow requests, scan status NoActivateControls
SteveTownsend commented 8 months ago

Split the check for OK-to-scan into two - OK-to-activate (fine grain) and OK-to-scan, and applied logic changes as appropriate wherever the old brute force check was used