search

T-Troll / alienfx-tools

Alienware systems lights, fans, and power control tools and apps
MIT License
458 stars 39 forks source link

Unable to update from 8.0.0 #333

Closed XRone99 closed 1 year ago

XRone99 commented 1 year ago

Everytime I try to upgrade version from 8.0.0 to any other more recent version the new version crashes as soon as I open it. There's not any kind of error displayed on screen, the program just closes itself.

System Information

Additional context We already got in touch under 8.2.5 version comment thread, this is the latest backup I made (06/04/2023) alienfx-tools 8.0.0 backup.zip

T-Troll commented 1 year ago

Here are debug version for you - alienfx-gui.zip You will need Microsoft tool named DbgView. Run DbgView (from admin), then run app (from admin). Please share output from DbgView.

XRone99 commented 1 year ago

I just did what you asked this is the output LORENZO-ALIEN.LOG

T-Troll commented 1 year ago

Oh, i see. Interesting.... It's somewhere into effects. Let me check and think.

00000037    15.83634281 [23488] Second Chance Assertion Failed: File    
00000038    15.83645248 [23488] minkernel\crts\ucrt\inc\corecrt_internal_string_templates.h 
00000039    15.83657265 [23488] , Line  
00000040    15.83659649 [23488] 81

This one. After profile switch.

XRone99 commented 1 year ago

There's something else I could do to help debug?

T-Troll commented 1 year ago

I share new debug version for check today.

T-Troll commented 1 year ago

Here is new debug version, please run the same way (i need DbgView log) - alienfx-gui.zip

I've added some debug info to pinpoint the fault point, and also some checks. Seems like an issue somewhere into APIv5 (keyboard) control code.

XRone99 commented 1 year ago

LORENZO-ALIEN.LOG

Here's the log, program crashed ad always Let me know if I can do anything else to help

T-Troll commented 1 year ago

A mystery. Good news it's not APIv5, it's working well. Something wrong with string creation, some entity has forbidden name... Ok, let me check you config.

XRone99 commented 1 year ago

May I send you some file containing config information?

T-Troll commented 1 year ago

First, i not sure your current config is the same as your backup before (it's saved then elevation prompt appear). Anyway, it should not be this.

Here is another debug - alienfx-gui.zip - please check it the same way.

In case it crash, try to check some other ways:

PS: I see you have Gamma correcton disabled. Well... It will make lights a little brighter at the center colors, but it will not fit screen colors in many cases.

XRone99 commented 1 year ago

backup.zip LORENZO-ALIEN.LOG LORENZO-ALIEN-EditedREG.LOG

Before editing register I produced the first log and the program crashed as always, after that I edited register as you said and the program worked (second log)

UPD: Program worked for some minutes then I minimized it and it crashed and now it won't open ot it starts just to crash after some seconds Screenshot 2023-04-13 082127

T-Troll commented 1 year ago

Thanks for info, we are close to catch it.

T-Troll commented 1 year ago

So... I check error in logs - crash happened in string template, then one string copied to another. ... and getting out of space. So it can be non-terminated string or out of memory.

Seems like something wrong with the combination of the one name into config (but it looks ok) and your locale (Italian, i guess...). I use different locale(s), and your configuration works well for me. NB: It's not a good idea, in general, using national chars into names, app is short-locale, not unicode to decrease size.

It's a pity i forget to ask you to press "Save" button to rewrite converted configuration as far as app start, but it's ok.

There are many points you can get incorrect string, let us check it one-by-one (you have config backups, so can restore all back any time):

Use latest debug version i provide (release doesn't have support of old zones from your config). I don't need more logs for now.

PS: You don't have ESIF sensor names from DPTF, and use old one. If you want to fill it (one-time operation), copy 2 files from app to system folder (refer readme for names and path), change HKCU/Alienfan/DPTF to 1, and run fx or fan gui. Start will be delayed for 3-5 seconds, but names will be filled by real one from ESIF #N.

XRone99 commented 1 year ago

Followed the first point of your list (deleted AlienfxGUI key) and the debug version you sent me worked fine until I minimized it Meanwhile I set up some setting manually as they were before the update Now after some attempts to start it the situation returned to previous (so it doesn't even start and crushes diractly) I wonder if there's something with te fact that the program starts minimized... By the way here's log. Sorry for late reply but in those days I've been away from home LORENZO-ALIEN 3.LOG

T-Troll commented 1 year ago

Hmm... Going minimized, just unload active UI and start auto profile switch. But according to delay between refresh and exception, this can be other.

So let's make the last effort - alienfx-gui.zip Try this version (i don't need logs if it crashed, it will be the same).

We have crazy situation - your configuration works well at my gear, and some other guys didn't complain about crashes on the same gear as you have. According to exception, it looks like a combination of settings + your locale.

So... For next steps, we only have a choice of the remote debug at your machine. Contact me in Discord (link to the server at project readme) about it.

XRone99 commented 1 year ago

Ok, I just downloaded this version:

  1. It seems to work but it erased all the settings of ambient lighting (it's not a problem i'll set them back)
  2. Now I'm going to set up manually my previous setup and see if it makes app crash
  3. Set up ambient lighting in a couple of profiles and still seems to work properly
  4. Going to reboot and see if it starts properly
  5. System rebooted normally and AlienFX GUI started with windows as expected Everything seems to work pretty good, I don't know what you did but I think we can say the issue has been fixed
T-Troll commented 1 year ago

I'm glad to hear it, but hold on! All i do.... remove tray tooltip. Seems like it provide crash on your settings then generates tool tip text. The good question is why, so we need to investigate it.

XRone99 commented 1 year ago

What can I do to help?

(Besides i found something that could be a little bug: i created 3 profiles to provide different lighting and fan behaviour in different situation but for some reason 2 of these are bond; if I modify Fan profile on one the other gets modified too, I don't know if it's some kind of bond setting or it's a real bug)

T-Troll commented 1 year ago

Well... Here are release with some changes into tooltip generation. Please check it - alienfx-gui.zip

It's two possible broke point inside it, this is the check for one (and new format as well).

About profiles, make sure you have "Fan settings" selected for profile at "Profiles" tab, or it will share the same system fan profile. If you check it, a copy of system fan profile will be copied, and you can edit and apply it separately. NB: You will edit system fan or profile fan regarding of this setting and what profile selected into top UI line.

PS: I found a bug into Windows. Then i add Italian locale for testing, it also installs Italian keyboard unexpectedly. And it can't be removed! (well... i remove it after some effort). I report this to M$.

XRone99 commented 1 year ago

This version seems to work well, here's log LORENZO-ALIEN 4.LOG

About fan issue the situation fixed itself for some reason ("Fan setting" was already selected on each of the 3 profiles I have)

T-Troll commented 1 year ago

Glad to hear it. So... I can tell you what happened. You config provides a moment you have no active profile at all, and W11 ask to generate tooltip at this time. I didn't check this situation before, so app crashes.

I think fan was not an issue, you just edit incorrect profile or profile without "Fan settings" active.

The version i send is a release one, so you can continue to use it. I'll patch it at next release as well.

XRone99 commented 1 year ago

Thank you for support, I'm happy to have been of help