- Pharo-dev - lists.pharo.org

Re: [Pharo-project] Wrong number with message tally?

by StÃ©phane Ducasse

Alex normally I integrated the fixes of juan long time ago. Now could you have a look at CUIS to see if you get the same effect. Stef On Dec 24, 2011, at 3:29 PM, Alexandre Bergel wrote: >> This means that some other process (probably the finalization process, since #factorial creates a lot of garbage) was running and/or gc happened during the "missing" 34.2%. The gc times are listed below. > > I also thought about that. But I doubt this is the only reason. I tried: > > MessageTally spyOn: [ 100 timesRepeat: [XMLDOMParser parse: (FileStream fileNamed: 'path to an xml file') contents ]]. > > and I obtain > -=-=-=-=-=-=-=-=-=-=-=-= > - 168 tallies, 168 msec. > > **Tree** > -------------------------------- > Process: (40s) 754188288: nil > -------------------------------- > 18.5% {31ms} XMLDOMParser>>parseDocument > 18.5% {31ms} XMLDOMParser(SAXHandler)>>parseDocument > 18.5% {31ms} XMLParser>>parseDocument > 18.5% {31ms} XMLParser>>parseToken > 18.5% {31ms} XMLTokenizer>>nextToken > 16.1% {27ms} XMLTokenizer>>nextMarkupToken > -=-=-=-=-=-=-=-=-=-=-=-= > > The memory stat are > -=-=-=-=-=-=-=-=-=-=-=-= > > **Memory** > old +120,524 bytes > young -442,960 bytes > used -322,436 bytes > free +322,436 bytes > > **GCs** > full 0 totalling 0ms (0.0% uptime) > incr 12 totalling 5ms (3.0% uptime), avg 0.0ms > tenures 2 (avg 6 GCs/tenure) > root table 0 overflows > -=-=-=-=-=-=-=-=-=-=-=-= > > > The factorial example consumes more memory, and it is reported to consume 72% > > Something is weird. I will look into it. > > Cheers, > Alexandre > -- > _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: > Alexandre Bergel http://www.bergel.eu > ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. > > > > > >

Dec. 24, 2011

Re: [Pharo-project] Good practicces! Looking for examples

by StÃ©phane Ducasse

On Dec 24, 2011, at 3:07 PM, Alexandre Bergel wrote: > Not sure what you mean with good coding practice, but here my contrib: > > - the command pattern (e.g., OB, Mondrian Easel, MonticelloBrowser) > - symbol polymorphic to block and other object-as-function Not sure that this is a good practices. I think that you like them because you use Smalltalk as a DSL. > - writing test before fixing code > - having short methods indeed Here is what I started to write.

Dec. 24, 2011

Re: [Pharo-project] Wrong number with message tally?

by Alexandre Bergel

> This means that some other process (probably the finalization process, since #factorial creates a lot of garbage) was running and/or gc happened during the "missing" 34.2%. The gc times are listed below. I also thought about that. But I doubt this is the only reason. I tried: MessageTally spyOn: [ 100 timesRepeat: [XMLDOMParser parse: (FileStream fileNamed: 'path to an xml file') contents ]]. and I obtain -=-=-=-=-=-=-=-=-=-=-=-= - 168 tallies, 168 msec. **Tree** -------------------------------- Process: (40s) 754188288: nil -------------------------------- 18.5% {31ms} XMLDOMParser>>parseDocument 18.5% {31ms} XMLDOMParser(SAXHandler)>>parseDocument 18.5% {31ms} XMLParser>>parseDocument 18.5% {31ms} XMLParser>>parseToken 18.5% {31ms} XMLTokenizer>>nextToken 16.1% {27ms} XMLTokenizer>>nextMarkupToken -=-=-=-=-=-=-=-=-=-=-=-= The memory stat are -=-=-=-=-=-=-=-=-=-=-=-= **Memory** old +120,524 bytes young -442,960 bytes used -322,436 bytes free +322,436 bytes **GCs** full 0 totalling 0ms (0.0% uptime) incr 12 totalling 5ms (3.0% uptime), avg 0.0ms tenures 2 (avg 6 GCs/tenure) root table 0 overflows -=-=-=-=-=-=-=-=-=-=-=-= The factorial example consumes more memory, and it is reported to consume 72% Something is weird. I will look into it. Cheers, Alexandre -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.

Dec. 24, 2011

Re: [Pharo-project] Good practicces! Looking for examples

by Alexandre Bergel

Not sure what you mean with good coding practice, but here my contrib: - the command pattern (e.g., OB, Mondrian Easel, MonticelloBrowser) - symbol polymorphic to block and other object-as-function - writing test before fixing code - having short methods Alexandre On 24 Dec 2011, at 10:53, StÃ©phane Ducasse wrote: > Hi guys > > talking with sven during the last sprint we realized that it would be good to collect and document some good practices. > So I started a chapter for PBE2 on coding practices. > > I will take the some of the optimization that sven did in time/date printOn: as a start > > If you have some suggestions with examples I could integrate to the chapter. > > > Stef > > > -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.

Dec. 24, 2011

[Pharo-project] Good practicces! Looking for examples

by StÃ©phane Ducasse

Hi guys talking with sven during the last sprint we realized that it would be good to collect and document some good practices. So I started a chapter for PBE2 on coding practices. I will take the some of the optimization that sven did in time/date printOn: as a start If you have some suggestions with examples I could integrate to the chapter. Stef

Dec. 24, 2011

[Pharo-project] [update 1.4] #14264

by StÃ©phane Ducasse

14264 ----- - Issue 5118: Check pointsTo: and InstVarInclude: in squeak. Thanks levente and mariano. http://code.google.com/p/pharo/issues/detail?id=5118 - Issue 5123: Clone is sent but not defined in FloatArray http://code.google.com/p/pharo/issues/detail?id=5123 - Issue 5112: More consistent SharedPool protocol. Thanks Mariano. http://code.google.com/p/pharo/issues/detail?id=5112 Stef (more to come)â¦but this is sunny today :)

Dec. 24, 2011

Re: [Pharo-project] gtinspector

by Mariano Martinez Peck

Thanks Sean. That's much more reasonable :) So ... I will put back GTInspector to my images by loading exactly that. On Fri, Dec 23, 2011 at 11:07 PM, Sean P. DeNigris <sean(a)clipperadams.com>wrote: > > Mariano Martinez Peck wrote > > > > If I just want to install GTInspector > > > > > For now (works in 1.3)... > Gofer new > squeaksource: 'Glamour'; > package: 'ConfigurationOfGlamour'; > load. > (ConfigurationOfGlamour project version: 'default') load: {'Core'. > 'Morphic'}. > Gofer new > squeaksource: 'glamoroust'; > package: 'GT-Inspector'; > load. > > -- > View this message in context: > http://forum.world.st/gtinspector-tp4220848p4229945.html > Sent from the Pharo Smalltalk mailing list archive at Nabble.com. > > -- Mariano http://marianopeck.wordpress.com

Dec. 23, 2011

Re: [Pharo-project] gtinspector

by Sean P. DeNigris

Mariano Martinez Peck wrote > > If I just want to install GTInspector > For now (works in 1.3)... Gofer new squeaksource: 'Glamour'; package: 'ConfigurationOfGlamour'; load. (ConfigurationOfGlamour project version: 'default') load: {'Core'. 'Morphic'}. Gofer new squeaksource: 'glamoroust'; package: 'GT-Inspector'; load. -- View this message in context: http://forum.world.st/gtinspector-tp4220848p4229945.html Sent from the Pharo Smalltalk mailing list archive at Nabble.com.

Dec. 23, 2011

Re: [Pharo-project] gtinspector

by Tudor Girba

Hi, Yes, you can. It's just that the configuration of Glamoroust is not made for that. At this moment, the only way to do it is by loading the package: Gofer new squeaksource: 'glamoroust'; package: 'GT-Inspector'; load. Cheers, Doru On 23 Dec 2011, at 14:27, Sean P. DeNigris wrote: > > Tudor Girba-2 wrote >> >> Yes. It will load: Glamour (which loads Mondrian, EyeSee, Magritte and >> some of its unnecessary dependencies :)) > > Is it possible to load the toolkit on top of basic Glamour (w/o Mondrian, > EyeSee, Magritte...)? > > -- > View this message in context: http://forum.world.st/gtinspector-tp4220848p4228581.html > Sent from the Pharo Smalltalk mailing list archive at Nabble.com. > -- www.tudorgirba.com "We can create beautiful models in a vacuum. But, to get them effective we have to deal with the inconvenience of reality."

Dec. 23, 2011

Re: [Pharo-project] Wrong number with message tally?

by Alexandre Bergel

It is not supposed to do: -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= MessageTally class>>spyOn: aBlock " Spy on aBlock, in the current process. Can include or not statistics on other processes in the report. [1000 timesRepeat: [ 100 timesRepeat: [120 factorial]. (Delay forMilliseconds: 10) wait ]] forkAt: 45 named: '45'. MessageTally spyOn: [10000 timesRepeat: [1.23 printString]] " ^self spyOn: aBlock reportOtherProcesses: false -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= I am not very familiar with the threading mechanism in Pharo, but since factorial does not explicitly give the hand to other threads, how can other threads be activated while the factoring is being computed? Cheers, Alexandre On 23 Dec 2011, at 14:12, Nicolas Cellier wrote: > Doesn't it tally several processes ? (finalization etc...) > > Nicolas > > 2011/12/23 Alexandre Bergel <alexandre.bergel(a)me.com>: >> Hi! >> >> MessageTally seems to give wrong values regarding the relative time spent. Consider: >> MessageTally spyOn: [ 100 timesRepeat: [ 1000 factorial ] ] >> >> It gives something like: >> -=-=-=-=-=-=-=-=-=-=-=-= >> - 377 tallies, 379 msec. >> >> **Tree** >> -------------------------------- >> Process: (40s) 464519168: nil >> -------------------------------- >> 65.8% {249ms} SmallInteger(Integer)>>factorial >> 65.8% {249ms} SmallInteger(Integer)>>factorial >> 65.8% {249ms} SmallInteger(Integer)>>factorial >> 65.8% {249ms} SmallInteger(Integer)>>factorial >> ... >> -=-=-=-=-=-=-=-=-=-=-=-= >> >> I would expect to have 100% at the root of the call graph since other threads are not reported. >> >> Is there something I am missing? If no, then I will dive into MessageTally. This is annoying. >> >> Cheers, >> Alexandre >> >> -- >> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >> Alexandre Bergel http://www.bergel.eu >> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >> >> >> >> >> >> > -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.

Dec. 23, 2011