Morph21 / MercuryTrade-Community-Fork

An overlay application for Path of Exile(Tracking, Trading, Chat, etc)
MIT License
142 stars 23 forks source link

[BUG] - huge cpu +ram usage when notification window pop up #109

Closed tvinki899 closed 2 years ago

tvinki899 commented 2 years ago

изображение

i try enable\disable fore\background fps cap, change numbers .jar or .exe reinstall java clear config folder = still same usage log clear

Morph21 commented 2 years ago

Does it happens even for test notifications?

tvinki899 commented 2 years ago

изображение if u about this, when i clicked settings window close and nothing happend i think something wrong in my current system

Morph21 commented 2 years ago

Yeah, that's weird. Can you run Mercury trade from CMD and try to open tests once more? You should see any errors in CMD then.

You just need to open CMD inside folder with MercuryTrade.jar the type in console:

java -jar MercuryTrade.jar

And paste any errors here, I will look into it (maybe it's some problem with java version) Also do you have newest MT version?

tvinki899 commented 2 years ago

an its from new installed windows 10 same problem 25% CPU

C:\Users\Admin\Downloads\MercuryTrade.jar\MercuryTrade>java -jar MercuryTrade.jar

JNativeHook: Global keyboard and mouse hooking for Java.
Copyright (C) 2006-2015 Alexander Barker.  All Rights Received.
https://github.com/kwhat/jnativehook/

JNativeHook is free software: you can redistribute it and/or modify
it under the terms of the GNU Lesser General Public License as published
by the Free Software Foundation, either version 3 of the License, or
(at your option) any later version.

JNativeHook is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
GNU General Public License for more details.

You should have received a copy of the GNU Lesser General Public License
along with this program.  If not, see <http://www.gnu.org/licenses/>.
Morph21 commented 2 years ago

Can you try running MercuryTrade.jar from cmd with this parameters and check if it solves problem with memory usage?

java -Xmx256m -XX:+UseG1GC -jar MercuryTrade.jar

-Xmx256m - should limit the memory usage -XX:+UseG1GC - should result in less cpu used by garbage collector and faster clean up of memory

sarathustra commented 2 years ago

i have same issue and it was since begin, thats why i comeback to original mercury

Morph21 commented 2 years ago

@sarathustra can you try to run newest version with .exe file? It should solve the memory issue. Also original mercury I'm pretty sure won't show notifications

sarathustra commented 2 years ago

@sarathustra can you try to run newest version with .exe file? It should solve the memory issue. Also original mercury I'm pretty sure won't show notifications

it works without problem before this league, now its bugged, but fork mercury bugged aswell, 2-3 pms and then it stop works

Morph21 commented 2 years ago

@sarathustra

  1. which version of MercuryTrade-community-fork did you use?
  2. Did you run it from exe file or from jar file?
  3. Are there any errors in logs folder?
  4. Which java version do you use?
sarathustra commented 2 years ago
  1. last
  2. both
  3. https://pastebin.com/1bd3nxdb
  4. build 1.8.0_281-b09
Morph21 commented 2 years ago

@sarathustra It looks like totally different issue than te OP created.

I will look into it.

In the meantime clean installation of mercury trade should fix that issue

By clean I mean that you need to delete C:/Users/your_username/AppData/Local/MercuryTrade

Before deletion you can backup configuration file inside folder C:/Users/your_username/AppData/Local/MercuryTrade It contains all settings.

I hope that it helps untill I will solve that issue

Morph21 commented 2 years ago

Fixed with https://github.com/Morph21/MercuryTrade-Community-Fork/releases/tag/1.2.3