ralsina / marave

Automatically exported from code.google.com/p/marave
GNU General Public License v2.0
0 stars 2 forks source link

marave is really CPU-hungry #88

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. run marave and change background music
2. look at cpu usage in top or htop

What version of the product are you using? On what operating system?
I'm running marave-editor 0.7 beta, installed on Kubuntu 10.04 using 
easy_install marave. 

Please provide any additional information below.
I'm running it on a MacBook Pro 5.5, with a  intel Core 2 Duo @2.26 GHz. After 
some moments I started marave-editor, the CPU fan start to spin and CPU usage 
increase a lot. At the moment, htop says that the process is using ~60% of my 
CPU.

Original issue reported on code.google.com by gerlo...@gmail.com on 8 Jul 2010 at 12:33

GoogleCodeExporter commented 9 years ago
And without music that doesn't happen? How about with/without key clicks?

Original comment by roberto.alsina on 8 Jul 2010 at 1:06

GoogleCodeExporter commented 9 years ago
Ubuntu 10.10, marave-0.7. All the time program eat 100% cpu. Sound disabled.

Original comment by zob...@gmail.com on 21 Feb 2011 at 10:23

GoogleCodeExporter commented 9 years ago
This issue seems to be a problem with MainWidget.scenechanged. Near line 1330, 
the condition should be something like the code below. Otherwise it will always 
be true. This is what causes high cpu load.

# Bottom-Left corner dragged
[...]
if x != self.editorX-2*m or \
    y != self.editorY+self.editorH:
[...]

Btw, great editor!

Original comment by fnavar...@gmail.com on 27 Mar 2011 at 11:17

GoogleCodeExporter commented 9 years ago
fix?: 
https://github.com/koo5/yorave/commit/bd35a10c6f8c589f11f6ad3ec76c7ca9cbe23445

Original comment by running....@gmail.com on 6 Apr 2012 at 7:22

GoogleCodeExporter commented 9 years ago
fnavar...'s fix works for me (replace + with -).  Linked patch does NOT work 
for me with 0.7-6.fc16 on Fedora 16.

Original comment by aaron.hamid on 3 Oct 2012 at 1:10