karensg / crowd-summary

Crowd Summary Tool
0 stars 1 forks source link

Max execution time of script #37

Closed bouke-nederstigt closed 10 years ago

bouke-nederstigt commented 10 years ago

@MBrouns It seems the stanford classifier takes a bit more time. Should the script still wait for a response? and if so what should the max execution time be. 30s doesn't seem to cut it anymore.

MBrouns commented 10 years ago

I know it's a lot slower but it shouldn't take 30 seconds. could you try calling the jar from the command line to see how fast it is?

bouke-nederstigt commented 10 years ago

Timed about 50 sec running from the command line. May be the problem will be fixed if fulltext is inserted in the db properly? Or is this not used anymore? Also the program generated an error.

image

bouke-nederstigt commented 10 years ago

Setting higher execution time seems to solve the problem for now.

MBrouns commented 10 years ago

I haven't tried with as many documents as you have but with 10 docs it still only took about a second, maybe at most 3 seconds. Fulltext has to be filled in for all documents for it to work correctly though.

I'll look into the error

On Mon, Mar 31, 2014 at 10:11 AM, bouke-nederstigt <notifications@github.com

wrote:

Timed about 50 sec running from the command line. May be the problem will be fixed if fulltext is inserted in the db properly? Or is this not used anymore? Also the program generated an error.

[image: image]https://cloud.githubusercontent.com/assets/3441985/2564982/17b35f68-b8ac-11e3-8428-e706442c68ca.png

Reply to this email directly or view it on GitHubhttps://github.com/yetti4/crowd-summary/issues/37#issuecomment-39063223 .

bouke-nederstigt commented 10 years ago

Weird. It was just a few lorem ipsums as well. Not even 50 lines.

Bouke Nederstigt


Oude Delft 223 2611HD Delft

MOB: (+31) 65 34 47 826

2014-03-31 10:20 GMT+02:00 MBrouns notifications@github.com:

I haven't tried with as many documents as you have but with 10 docs it still only took about a second, maybe at most 3 seconds. Fulltext has to be filled in for all documents for it to work correctly though.

I'll look into the error

On Mon, Mar 31, 2014 at 10:11 AM, bouke-nederstigt < notifications@github.com

wrote:

Timed about 50 sec running from the command line. May be the problem will be fixed if fulltext is inserted in the db properly? Or is this not used anymore? Also the program generated an error.

[image: image]< https://cloud.githubusercontent.com/assets/3441985/2564982/17b35f68-b8ac-11e3-8428-e706442c68ca.png

Reply to this email directly or view it on GitHub< https://github.com/yetti4/crowd-summary/issues/37#issuecomment-39063223>

.

Reply to this email directly or view it on GitHubhttps://github.com/yetti4/crowd-summary/issues/37#issuecomment-39063897 .