- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Never ending BlockClosure>>benchFor:

by Martin Dias

I also tried with a modified #benchFor: that doesn't use a forked block, and that fixed the issue. benchFor: duration "Run me for duration and return a BenchmarkResult" "[ 100 factorial ] benchFor: 2 seconds" | count started endTime | count := 0. started := Time millisecondClockValue. endTime := started + duration asMilliSeconds. [ Time millisecondClockValue > endTime ] whileTrue: [ self value. count := count + 1 ]. ^ BenchmarkResult new iterations: count; elapsedTime: (Time millisecondsSince: started) milliSeconds; yourself Do you think comparing each time, instead of using #fork is a bad idea? That's how it was working #bench in Pharo 3. Martin On Thu, Nov 12, 2015 at 2:42 AM, Martin Dias <tinchodias(a)gmail.com> wrote: > Hello, > > I'm trying to understand why this method hangs in certain conditions*. > What happens is that the #whileTrue: in the following method is never cut. > > benchFor: duration > "Run me for duration and return a BenchmarkResult" > "[ 100 factorial ] benchFor: 2 seconds" > | count run started | > count := 0. > run := true. > [ duration wait. run := false ] forkAt: Processor timingPriority - 1. > started := Time millisecondClockValue. > [ run ] whileTrue: [ self value. count := count + 1 ]. > ^ BenchmarkResult new > iterations: count; > elapsedTime: (Time millisecondsSince: started) milliSeconds; > yourself > > Strangely, I verified with several #logCr that the forked block starts, > but the #wait never ends. > ---> Maybe you see something I don't in the code. Any clue? > > I will continue trying to isolate the problem, but maybe you have some > idea. > > * the conditions to reproduce are: > 1) curl get.pharo.org/50+vm | bash > 2) ./pharo Pharo.image get Epicea 6.5 > 3) ./pharo Pharo.image test '^(?!Metacello)[A-L].*' > > Notes: > - It doesn't hang when running from UI > - it doesn't hang when running less tests, like: > pharo Pharo.image test '^(?!Metacello)[E-L].*' > - it *does* hang even when removing all Epicea tests > - Epicea doesn't extend or override "kernel" behavior... > > Regards, > Martin >

Nov. 12, 2015

Never ending BlockClosure>>benchFor:

by Martin Dias

Hello, I'm trying to understand why this method hangs in certain conditions*. What happens is that the #whileTrue: in the following method is never cut. benchFor: duration "Run me for duration and return a BenchmarkResult" "[ 100 factorial ] benchFor: 2 seconds" | count run started | count := 0. run := true. [ duration wait. run := false ] forkAt: Processor timingPriority - 1. started := Time millisecondClockValue. [ run ] whileTrue: [ self value. count := count + 1 ]. ^ BenchmarkResult new iterations: count; elapsedTime: (Time millisecondsSince: started) milliSeconds; yourself Strangely, I verified with several #logCr that the forked block starts, but the #wait never ends. ---> Maybe you see something I don't in the code. Any clue? I will continue trying to isolate the problem, but maybe you have some idea. * the conditions to reproduce are: 1) curl get.pharo.org/50+vm | bash 2) ./pharo Pharo.image get Epicea 6.5 3) ./pharo Pharo.image test '^(?!Metacello)[A-L].*' Notes: - It doesn't hang when running from UI - it doesn't hang when running less tests, like: pharo Pharo.image test '^(?!Metacello)[E-L].*' - it *does* hang even when removing all Epicea tests - Epicea doesn't extend or override "kernel" behavior... Regards, Martin

Nov. 12, 2015

[pharo-project/pharo-core] a32fc8: 40625

by GitHub

Branch: refs/heads/4.0 Home: https://github.com/pharo-project/pharo-core Commit: a32fc885d8d2382720ab781ddf45f13ccdc5cd47 https://github.com/pharo-project/pharo-core/commit/a32fc885d8d2382720ab781d… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-11-11 (Wed, 11 Nov 2015) Changed paths: M Kernel.package/Object.class/instance/-- all --/mustBeBooleanInMagic_.st M OpalCompiler-Core.package/IRBytecodeGenerator.class/instance/-- all --/bytecodes.st M OpalCompiler-Core.package/IRBytecodeGenerator.class/instance/results/bytecodes.st M OpalCompiler-Core.package/extension/Object/instance/mustBeBooleanInMagic_.st A OpalCompiler-Tests.package/MustBeBooleanTests.class/instance/-- all --/testDNU.st A OpalCompiler-Tests.package/MustBeBooleanTests.class/instance/tests/testDNU.st M ScriptLoader40.package/ScriptLoader.class/instance/-- all --/commentForCurrentUpdate.st R ScriptLoader40.package/ScriptLoader.class/instance/-- all --/script624.st A ScriptLoader40.package/ScriptLoader.class/instance/-- all --/script625.st R ScriptLoader40.package/ScriptLoader.class/instance/-- all --/update40624.st A ScriptLoader40.package/ScriptLoader.class/instance/-- all --/update40625.st R ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script624.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script625.st R ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40624.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40625.st M ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 40625 16908 wrong source node for pc in mustBeBooleanInMagic https://pharo.fogbugz.com/f/cases/16908 http://files.pharo.org/image/40/40625.zip

Nov. 11, 2015

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/40625 Home: https://github.com/pharo-project/pharo-core

Nov. 11, 2015

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/50436 Home: https://github.com/pharo-project/pharo-core

Nov. 11, 2015

[pharo-project/pharo-core] f7c3ed: 50436

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: f7c3eda3845b5c88c94b76edc7674fbd58d0b647 https://github.com/pharo-project/pharo-core/commit/f7c3eda3845b5c88c94b76ed… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-11-11 (Wed, 11 Nov 2015) Changed paths: M Reflectivity.package/ReflectiveMethod.class/instance/invalidate/metaLinkOptions.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50435.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50436.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50435.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50436.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Traits.package/TBehavior.class/instance/adding%2Fremoving methods/addSelectorSilently_withMethod_.st Log Message: ----------- 50436 16991 fix: be able to invalidate the whole image (classes and metaclasses) https://pharo.fogbugz.com/f/cases/16991 16985 #methodDictAddSelectorSilently:withMethod: not needed https://pharo.fogbugz.com/f/cases/16985 http://files.pharo.org/image/50/50436.zip

Nov. 11, 2015

Smalltalkhub returning 500 response for a public repository

by Euan Mee

I've been trying unsuccessfully to write to the SandstoneDb public repository at smalltalkhub.com http://smalltalkhub.com/#!/~gnaritas/SandstoneDb shows the green tickmark and PUBLIC Writing to a package within it receives a 500 error. The specific package I am writing to is: SandstoneDbTests which is marked in Monticello (after I have downloaded it) as belonging in http://smalltalkhub.com/mc/gnaritas/SandstoneDb/main as do all the other listed packages, i.e. ConfigurationOfSandstoneDb SandstoneDb SandstoneDbFileSystemStore SandstoneDbFuel SandstoneDbFileDirectoryStore SandstoneDbSqueak SandstoneGOODS SandstondDbGOODSTests (Thank you to Max Leske for confirming it was a 500 error at the smalltalkhub.com end that was causing the debugger to respond to my attempted commit).

Nov. 11, 2015

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/50435 Home: https://github.com/pharo-project/pharo-core

Nov. 11, 2015

[pharo-project/pharo-core] f98ef7: 50435

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: f98ef74799ccfb6cba3e829100ce25ff26833635 https://github.com/pharo-project/pharo-core/commit/f98ef74799ccfb6cba3e8291… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-11-11 (Wed, 11 Nov 2015) Changed paths: M Reflectivity.package/ReflectiveMethod.class/instance/invalidate/installCompiledMethod.st A Reflectivity.package/ReflectiveMethod.class/instance/invalidate/installMethod_.st A Reflectivity.package/ReflectiveMethod.class/instance/invalidate/installReflectiveMethod.st M Reflectivity.package/extension/CompiledMethod/instance/invalidate.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50434.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50435.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50434.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50435.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Tool-Base.package/AbstractTool.class/instance/class/compileANewClassFrom_notifying_startingFrom_.st Log Message: ----------- 50435 16990 Class definition got broken https://pharo.fogbugz.com/f/cases/16990 16989 simplify installing reflective method https://pharo.fogbugz.com/f/cases/16989 http://files.pharo.org/image/50/50435.zip

Nov. 11, 2015

Re: [Pharo-dev] MNU RBBroweserEnvironment>>includesKey:

by stepharo

fix is getting processed now. Le 11/11/15 18:14, stepharo a Ã©crit : > you are welcome :) > > > Le 11/11/15 17:59, Dimitris Chloupis a Ã©crit : >> thank you Stef >> >> On Wed, Nov 11, 2015 at 6:51 PM stepharo <stepharo(a)free.fr> wrote: >> >> this is probably linked to the changes denis did to remove Smalltalk >> globals everywhere. >> indeed this is it >> >> envt := Smalltalk globals. >> >> got changed into self browsedEnvironment but it is wrong since >> Nautilus >> defines >> >> AbstractNautilusUI>>browsedEnvironment >> >> ^ self model browsedEnvironment >> >> >> I will send a fix now >> >> Stef >> >> >> Le 11/11/15 09:09, Thierry Goubier a Ã©crit : >> > Le 11/11/2015 08:53, Dimitris Chloupis a Ã©crit : >> >> And I am back with new problems with pharo 5 , I hope you >> missed me :D >> >> >> >> If I try to create a new class called ChrTimeInputM that >> subclasses >> >> TextMorph I am getting this MNU with the stack bellow >> > >> > Strange. >> > >> > Code failing is this: >> > >> > ((oldClass isNil or: [oldClass theNonMetaClass name asString ~= >> > newClassName]) >> > and: [envt includesKey: newClassName asSymbol]) ifTrue: >> > ["Attempting to define new class over existing one when >> > not looking at the original one in this browser..." >> > (self confirm: ((newClassName , ' is an existing >> class in >> > this system. >> > Redefining it might cause serious problems. >> > Is this really what you want to do?') asText makeBoldFrom: 1 to: >> > newClassName size)) >> > ifFalse: [^ nil]]. >> > >> > The strange thing is that, of anything which looks like an >> environment >> > in Pharo, very few do have an includesKey: and certainly not >> > RBBrowserEnvironment >> > >> > Moreover, in the case of scoped browsing, I don't think the test >> > amount to anything significant: if I try to redefine Object and if >> > Object is not in the browsed environment, I'll get no warning >> whatsoever. >> > >> > Thierry >> > >> >> I tried to replace includesKey: with includesClass: but then >> it alerts >> >> me falsely that there is already an existing class in the >> image, I tried >> >> this on the latest pharo 5 image with nothing installed in it. >> Here is >> >> the stack >> >> >> >> RBBrowserEnvironment(Object)>>doesNotUnderstand: #includesKey: >> >> >> PackageTreeNautilusUI(AbstractTool)>>compileANewClassFrom:notifying:startingFrom: >> >> >> >> >> PackageTreeNautilusUI(AbstractNautilusUI)>>compileAClassFrom:notifying: >> >> ClassDefinitionAcceptor>>accept:notifying: >> >> PackageTreeNautilusUI(NautilusUI)>>compileSource:notifying: >> >> PackageTreeNautilusUI(AbstractNautilusUI)>>whenTextAcceptedInView: >> >> MessageSend>>value: MessageSend>>cull: MessageSend>>cull:cull: >> [ action >> >> cull: anAnnouncement cull: announcer ] in >> >> AnnouncementSubscription>>deliver: in Block: [ action cull: >> >> anAnnouncement cull: announcer ] BlockClosure>>on:do: >> >> BlockClosure>>on:fork: AnnouncementSubscription>>deliver: [ >> subscription >> >> deliver: anAnnouncement ] in >> >> SubscriptionRegistry>>deliver:to:startingAt: in Block: [ >> subscription >> >> deliver: anAnnouncement ] BlockClosure>>ifCurtailed: >> >> SubscriptionRegistry>>deliver:to:startingAt: >> >> SubscriptionRegistry>>deliver:to: SubscriptionRegistry>>deliver: >> >> Announcer>>announce: RubScrolledTextMorph>>acceptContents >> >> RubScrolledTextMorph>>whenTextAcceptRequest: MessageSend>>value: >> >> MessageSend>>cull: MessageSend>>cull:cull: [ action cull: >> anAnnouncement >> >> cull: announcer ] in AnnouncementSubscription>>deliver: in >> Block: [ >> >> action cull: anAnnouncement cull: announcer ] BlockClosure>>on:do: >> >> BlockClosure>>on:fork: AnnouncementSubscription>>deliver: [ >> subscription >> >> deliver: anAnnouncement ] in >> >> SubscriptionRegistry>>deliver:to:startingAt: in Block: [ >> subscription >> >> deliver: anAnnouncement ] BlockClosure>>ifCurtailed: >> > >> > >> > >> >> >

Nov. 11, 2015