Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Brackets version 27 halt on Windows 7 #4400

Closed
hectorfhurtado opened this issue Jul 9, 2013 · 24 comments
Closed

Brackets version 27 halt on Windows 7 #4400

hectorfhurtado opened this issue Jul 9, 2013 · 24 comments

Comments

@hectorfhurtado
Copy link

Hi,

First of all, thanks for this awsome editor.

Second of all, with this new release ( 27 ), from time to time Brackets just halts and the content panel is turned into a light blue color and no button has effect, not even the red top right button to close it, I have to force the exit on Task manager. I don't know how this is triggered, It just happens.

I'm running Brackets on a Windows 7 machine.

This is a screenshot:

image

Thanks in advance.

@WebsiteDeveloper
Copy link
Contributor

@hectorfhurtado i also ran into this issue today.
I noticed that in the moment, when brackets starts to hang some brackets process just used an awfull lot of memory. But i couldn't reproduce the error at the moment. The only thing i noticed was, that the Brackets process seemed to use much memory, which increased with nearly every file i opened.

@hectorfhurtado
Copy link
Author

Hi, oh yes, when I just opened brackets, It consumed 57MB, but, when I started changed between files ( with Ctrl + Tab ), memory consumptions went to 125MB in 10 seconds more or less. I had opened 6 files and I was looping on these.

@JeffryBooher
Copy link
Contributor

High Priority. This isn't related to #4394.

@ghost ghost assigned JeffryBooher Jul 10, 2013
@njx
Copy link

njx commented Jul 10, 2013

@hectorfhurtado and @WebsiteDeveloper - this sounds like it might be the same issue as #3977. Are you using any particular large JS libraries (e.g. Angular) in the projects that you have open when Brackets hangs? If so, please let us know as that will help us narrow down the problem.

@ghost ghost assigned dangoor Jul 10, 2013
@njx
Copy link

njx commented Jul 10, 2013

High priority for sprint 28. Tentatively assigning to @dangoor since it sounds like these might be more Tern-related cases.

@cauliturtle
Copy link

@njx, I have similar problem too. I use Brackets to develop my company projects, which contains a large set of javascript file. Once updated to 27, It get around 10 seconds to open a file, any type of files.

update: not only open files, It needed to be wait also in switching the files on Working Files

@hectorfhurtado
Copy link
Author

HI, I'm using polymer and a bunch of node modules, but I've noticed that Brackets hangs when I have various opened files on my Working Files area and if I don't save on my current file and jump to another It just stops working. That exactly happend to me yesterday, very frustrating on that moment :) I'm back on release 26

@dangoor
Copy link
Contributor

dangoor commented Jul 11, 2013

From the look of it, I think you're right @njx. I'm going to start digging in now with one of the known failure cases.

@dangoor
Copy link
Contributor

dangoor commented Jul 12, 2013

@hectorfhurtado To help me reproduce this, would you mind telling me which Polymer files are in your project and what dependences/devDependencies you have in package.json for node? I'd like to try to set up something as similar as possible to see the behavior.

Here's what I have seen so far... I threw all of Polymer's .min.js files into a directory (just to be rough on the code hinting system). I installed express into node_modules, knowing that that comes along with a number of files (124 .js files in node_modules).

If I use cmd-tab to switch between files a bunch, the CPU works hard for a few seconds but the app never becomes unresponsive. I do see memory usage climbing as I do this. The brackets-helper process on my Mac is now at 738MB, up from 590MB before my last handful of cmd-tab switches. Given that each of the 6 Polymer files I have is more than 100K, this test is probably a bit harsher than reality, which is why I'm interested in knowing more specifically which libraries you have.

@hectorfhurtado
Copy link
Author

Hi, these are mi dependencies on node:

"dependencies": {
"express": "3.1.0",
"jade": "*",
"mongodb": "~1.2.0",
"socket.io": "~0.9.0",
"ssh2": "~0.2.5"
},
"devDependencies": {
"grunt": "~0.4.0",
"grunt-contrib-copy": "~0.4.1",
"grunt-contrib-concat": "~0.1.0",
"grunt-contrib-compass": "~0.1.0",
"grunt-contrib-coffee": "~0.7.0",
"grunt-contrib-livereload": "~0.1.0",
"grunt-contrib-uglify": "~0.2.0",
"grunt-contrib-watch": "~0.3.0",
"grunt-contrib-yuidoc": "~0.4.0",
}

From polymer, I'm using polymer.min.js and web-animations.js; but in total my project has 13.323 files, because I have a bunch of MIBs, scripts that act as agents, sass files, etc.

I don't know if Brackets actually tries to scan all of these files or just the ones I'm using on the working space. If Brackets is looking at all, is there a way to hide from brackets unrelevant files?

Thanks in advace

@hectorfhurtado
Copy link
Author

these are some detail of my machine:
Processor: Intel(R) COre(TM) i5-3320M CPU 2.60 GHz
RAM: 4.00 GB (2.89 GB usable)
System type: 32 bits OS

@RaymondLim
Copy link
Contributor

I can reproduce the hang with web-animations.js on my Windows 7 before getting changes from this pull request #4439. So I believe changes in Tern also fix the hang issue that can be caused by web-animations.js.

Update: Actually, I started out with ExtJS and switched to web-animation.js when I got the hang. So it may not be the web-animations.js.

@dangoor
Copy link
Contributor

dangoor commented Jul 13, 2013

@hectorfhurtado Brackets should scan just the .js files for code hinting purposes, which is what we're really looking at.

One more question: were you actively typing at the time the screen went blank? Or did this seem to happen without any specific action on your part?

If you spot this again, it would be interesting to note if you were doing something that triggers code hinting. For example, typing . or using "jump to definition". I noticed a problem with that on my Mac today, but it is not necessarily the same problem you're witnessing.

@hectorfhurtado
Copy link
Author

Hi, I was not only actively typing but switching back and forth between files without saving my work on them. I don't know if this added too much memory consumption. I triedon Firday to save my work every time and Brackets v27 didn't crashed.

Thanks in advance

@dangoor
Copy link
Contributor

dangoor commented Jul 15, 2013

Thanks for the update. The fact that you were bouncing back and forth without saving could be a useful clue.

@dangoor
Copy link
Contributor

dangoor commented Jul 16, 2013

I think this is likely the same as #4438 given that @hectorfhurtado's info gave me a scenario that lets me reliably reproduce that bug. More info to come.

@dangoor
Copy link
Contributor

dangoor commented Jul 16, 2013

Using Tern+CodeMirror on top of polymer and web-animations, you can see Chrome crash here:

(URL removed, because it's been reproduced enough)

@hectorfhurtado
Copy link
Author

Hi, Yes It crashes everytime on your demo.

I'm not using webAnimations.js anymore, I changed my code to use CSS3 animations instead because the new build of webAnimations of polymer stopped working and the new build lets you use CSS3 animations through polymer-scope. More interesting, when I removed from my code webAnimtaion.js my app now runs on Firefox ( before It only ran on Chrome ). Maybe that file was poisoned or something :)

Anyway, I've been using Brackets v27 since my last comment and It's been working well all the time.

Thanks in advance from preventing Turn turn crazy.

@dangoor
Copy link
Contributor

dangoor commented Jul 17, 2013

Great! I'm glad to hear that it's working better for you after you removed web-animations.js. I don't yet have a workaround or exact trigger isolated, so there's still a chance that the bug will crop up.

Given that I think your problem is the same as #4438, can I close this issue and we'll just use that other one to track this problem?

@hectorfhurtado
Copy link
Author

Hi, if #4438 is the same I think It's OK to close this issue.

On webAnimations.js seems to be a variable declaration to be undefined (Firefox console said) and I don't know if Turn is trying to resolve its value because that variable is used repeatedly.

Thanks again.

@dangoor
Copy link
Contributor

dangoor commented Jul 17, 2013

I changed my mind and decided that this issue has more useful information in it than #4438, so I'll keep this one open.

Thanks for the hint about the undefined variable. That may assist in finding a possible workaround.

@dangoor
Copy link
Contributor

dangoor commented Jul 22, 2013

I've posted a workaround for this crash. I would appreciate if anyone who is running into this crash could give that a try and let me know here if it works for you.

@dangoor
Copy link
Contributor

dangoor commented Jul 23, 2013

#4533 has landed. If you're running Brackets from master, please run git submodule sync and git submodule update and let me know if the fix is working for you.

@dangoor
Copy link
Contributor

dangoor commented Jul 23, 2013

Since this was my test case, I'm going to close this one. If anyone reading this spots another crash, please open a new issue and we'll track it down.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants