- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] 17240 Spur, when saving an image, it never shrinks

by Cyril Ferlicot D.

Le 17/02/2017 Ã 07:17, ClÃ©ment Bera a Ã©crit : > The full GC is indeed much faster as the previous compaction algorithm > was very slow. But unless you were executing memory intensive code in > very large heaps, you won't see much difference as full GC are uncommon. > We still need to make the GC incremental to avoid pauses ... > > > Can this fix the problem that MSE materialization takes 4 time longer with spur memory management? It would be really cool :) Do you have an idea of when the first incr GC without pause will be available? (No rush, it's just for info to know if it's in months or years :) ) -- Cyril Ferlicot http://www.synectique.eu 2 rue Jacques PrÃ©vert 01, 59650 Villeneuve d'ascq France

Feb. 17, 2017

[pharo-project/pharo-core]

by GitHub

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

Feb. 17, 2017

[pharo-project/pharo-core] e4d05f: 60398

by GitHub

Branch: refs/heads/6.0 Home: https://github.com/pharo-project/pharo-core Commit: e4d05f254c1c7b4c849437fc710eae435703bc91 https://github.com/pharo-project/pharo-core/commit/e4d05f254c1c7b4c849437fc… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2017-02-17 (Fri, 17 Feb 2017) Changed paths: A Kernel-Tests.package/ContextTest.class/README.md A Kernel-Tests.package/ContextTest.class/class/closures for testing/contextWithTempForTesting.st A Kernel-Tests.package/ContextTest.class/definition.st A Kernel-Tests.package/ContextTest.class/instance/private/expectedFailures.st A Kernel-Tests.package/ContextTest.class/instance/private/privRestartArgBlockTest.st A Kernel-Tests.package/ContextTest.class/instance/private/privRestartBlockArgsNoRemoteTempsTest.st A Kernel-Tests.package/ContextTest.class/instance/private/privRestartBlockTest.st A Kernel-Tests.package/ContextTest.class/instance/running/setUp.st A Kernel-Tests.package/ContextTest.class/instance/tests - ast mapping/testSourceNodeExecuted.st A Kernel-Tests.package/ContextTest.class/instance/tests - ast mapping/testSourceNodeExecutedWhenContextIsJustAtStartpc.st A Kernel-Tests.package/ContextTest.class/instance/tests - ast mapping/testSourceNodeOptimizedBlock.st A Kernel-Tests.package/ContextTest.class/instance/tests/testActivateReturnValue.st A Kernel-Tests.package/ContextTest.class/instance/tests/testCannotReturn.st A Kernel-Tests.package/ContextTest.class/instance/tests/testClosureRestart.st A Kernel-Tests.package/ContextTest.class/instance/tests/testFindContextSuchThat.st A Kernel-Tests.package/ContextTest.class/instance/tests/testJump.st A Kernel-Tests.package/ContextTest.class/instance/tests/testMethodContext.st A Kernel-Tests.package/ContextTest.class/instance/tests/testMethodIsBottomContext.st A Kernel-Tests.package/ContextTest.class/instance/tests/testReturn.st A Kernel-Tests.package/ContextTest.class/instance/tests/testSetUp.st A Kernel-Tests.package/ContextTest.class/instance/tests/testTempNamed.st A Kernel-Tests.package/ContextTest.class/instance/tests/testTempNamedPut.st A Kernel-Tests.package/ContextTest.class/instance/tests/verifyJumpWithSelector_.st R Kernel-Tests.package/MethodContextTest.class/README.md R Kernel-Tests.package/MethodContextTest.class/class/closures for testing/contextWithTempForTesting.st R Kernel-Tests.package/MethodContextTest.class/definition.st R Kernel-Tests.package/MethodContextTest.class/instance/private/expectedFailures.st R Kernel-Tests.package/MethodContextTest.class/instance/private/privRestartArgBlockTest.st R Kernel-Tests.package/MethodContextTest.class/instance/private/privRestartBlockArgsNoRemoteTempsTest.st R Kernel-Tests.package/MethodContextTest.class/instance/private/privRestartBlockTest.st R Kernel-Tests.package/MethodContextTest.class/instance/running/setUp.st R Kernel-Tests.package/MethodContextTest.class/instance/tests - ast mapping/testSourceNodeExecuted.st R Kernel-Tests.package/MethodContextTest.class/instance/tests - ast mapping/testSourceNodeExecutedWhenContextIsJustAtStartpc.st R Kernel-Tests.package/MethodContextTest.class/instance/tests - ast mapping/testSourceNodeOptimizedBlock.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testActivateReturnValue.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testCannotReturn.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testClosureRestart.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testFindContextSuchThat.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testJump.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testMethodContext.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testMethodIsBottomContext.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testReturn.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testSetUp.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testTempNamed.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/testTempNamedPut.st R Kernel-Tests.package/MethodContextTest.class/instance/tests/verifyJumpWithSelector_.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60397.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60398.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60397.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60398.st M ScriptLoader60.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 60398 15303 rename MethodContextTest to ContextTest https://pharo.fogbugz.com/f/cases/15303 http://files.pharo.org/image/60/60398.zip

Feb. 17, 2017

[pharo-project/pharo-core]

by GitHub

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

Feb. 17, 2017

[pharo-project/pharo-core] f71d05: 60397

by GitHub

Branch: refs/heads/6.0 Home: https://github.com/pharo-project/pharo-core Commit: f71d050f5e2884b2c26423b3ea93325544ab477c https://github.com/pharo-project/pharo-core/commit/f71d050f5e2884b2c26423b3… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2017-02-17 (Fri, 17 Feb 2017) Changed paths: A Morphic-Widgets-FastTable.package/FTPluggableIconListMorphAdaptor.class/instance/accessing/setMultipleSelection_.st M Random-Core.package/Random.class/instance/initialization/initialize.st A Random-Core.package/Random.class/instance/initialization/useClockBasedSeed.st A Random-Core.package/Random.class/instance/initialization/useUnixRandomGeneratorSeed.st M Random-Core.package/SharedRandom.class/README.md A Random-Tests.package/RandomTest.class/instance/tests/testDistribution.st A Random-Tests.package/RandomTest.class/instance/tests/testNextInt.st A Random-Tests.package/RandomTest.class/instance/tests/testNextInto.st A Random-Tests.package/RandomTest.class/instance/tests/testUnixRandomGeneratorSeed.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60396.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60397.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60396.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60397.st M ScriptLoader60.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st A Spec-Core.package/FastTableModel.class/class/example/exampleMultipleSelection.st M Spec-MorphicAdapters.package/MorphicFastTableAdapter.class/instance/factory/buildWidget.st M System-Changes.package/ChangeSet.class/instance/private/changeRecorderFor_.st M Tests.package/ChangeSetClassChangesTest.class/instance/testing/testAddInstanceVariableAddsNewChangeRecord.st M Tests.package/ChangeSetClassChangesTest.class/instance/testing/testChangeClassCategoryAddsNewChangeRecord.st Log Message: ----------- 60397 19715 FastTableModel: Cant enable multiple selection https://pharo.fogbugz.com/f/cases/19715 19211 sometimes I can not open changesort: receiver of endswith is nil https://pharo.fogbugz.com/f/cases/19211 19718 Give Random an option to initialize its seed from /dev/random https://pharo.fogbugz.com/f/cases/19718 http://files.pharo.org/image/60/60397.zip

Feb. 17, 2017

Re: [Pharo-dev] 2 seconds default time limit for tests

by stepharong

On Thu, 16 Feb 2017 14:46:41 +0100, Guillermo Polito <guillermopolito(a)gmail.com> wrote: > > > On Thu, Feb 16, 2017 at 2:31 PM, Denis Kudriashov <dionisiydk(a)gmail.com> > wrote: >> >> 2017-02-16 13:49 GMT+01:00 Guillermo Polito <guillermopolito(a)gmail.com>: >>> I vote for not introducing during code-freeze. >> >> I understand it but these behaviour was introduced in Pharo 6. And It >> is already used quite long time. So it could be considered as small >> improvements to already integrated feature. >>> >>> We can discuss it for Pharo 7, I'm not against actually. But I have >>> some concerns: >>> >>> - How does it work when we are debugging? >>> I mean, imagine I'm running a test, a debugger is open, and I start >>> working on it. How do these timeout currently work in this scenario? I >>> imagine that the test runner should detect you entered >>>into a "I'm >>> debugging" mode and should not kill your test, but this >> >> It solved very easily. "Watchdog process" just checks that test process >> isSuspended or not. If it is not suspended timeout is signalled. Test >> process is suspended only when debugged. > > Ok. Is something like this in a class comment? If not, we could add it, > because it will get lost in the mails. + 10000 > >> >>> is not clear to me how this works right now. >>> >>> - Also, how can we better document these things? Because I was bitten >>> a couple of times by this timeouts and the "I kill your process but I >>> do not tell you", and it was not nice because it was >>>completely >>> silent or obscure. >> >> Things which you show me I improved. Last days I fix Christophe issues >> which was really important. Anyway "fork management in tests" is not >> related to time limit of tests. And of course it is not >>supposed to >> make troubles but instead it should make life easy :) > > I know, I'm not pleading for improvements :). Just talking about > documenttion here. -- Using Opera's mail client: http://www.opera.com/mail/

Feb. 17, 2017

Re: [Pharo-dev] 2 seconds default time limit for tests

by Denis Kudriashov

I moved issue to Pharo 7 2017-02-16 13:33 GMT+01:00 Denis Kudriashov <dionisiydk(a)gmail.com>: > Hi. > > In Pharo 6 we introduced time limit for tests. Currently it is set to 1 > minute by default. And concrete test cases or single tests can redefine it > with suitable value. There is also setting to change default limit globally > which should help to not adopt external projects immediately if current > limit is bad for them. > > There is issue 19105 > <https://pharo.fogbugz.com/f/cases/19105/Default-time-limit-for-tests-should…> to > make limit really small. It will set default limit for 2 seconds. Slow > tests are already marked with bigger value. So it is safe for integration > > Here I ask for your agreement with this change. Or disagreement if there > are good reasons to not do this. > > I hope following story will convince you that default time limit should be > small: > > I worked on tests for new project and I got hanging tests. These tests > were waiting for some event which not happens because of bugs. They will be > failed after 1 minutes due to timeout. > > But when you work on code you of course do not want to wait 1 minute. So > it always forces manual interrupt of running tests. But then you could not > run full test suite to see all problem tests. And also it is not really > easy to detect current running test after interrupt. You need analyze stack > in debugger and if you just close debugger information is lost. > > So to fix it I change default timeout in each of my test cases. > If you will work on similar code you will need same approach to > comfortably detect and fix "synchronization" bugs. > > So this was my story. I am sure default behaviour with small time limit > will make SUnit much better. > Common case is that unit tests are supposed to be fast. Users expect it by > default. And if something is going wrong fast tests should fail fast. > When user wrote slow test it is expected to know that test is slow. And > for such rare cases user can mark slow tests explicitly. But SUnit should > not expect tests to be slow by default. > > I think it is really good improvement for Pharo 6. It makes TDD in Pharo > better. > > Best regards, > Denis >

Feb. 17, 2017

Re: [Pharo-dev] Code Completion in Pharo

by stepharong

Hi lukas Nice to see you here. I would start to look at the code and clean what is strange. - Did you test the difference between the two algorithms? - Did you play with NECPreferences? - I would like to see what is the difference between the NOC class and the NECclass - I would like to remove the link with Smalltalk tools - I would be nice to improve the class comments. - there are some tests for NEC but -- are there any good? Did you read them? - there are no tests for NOC -- it would be nice to add tests. Looking at the code quickFillWithCategoriesMatching: pattern (Smalltalk organization categoriesMatching: pattern) do: [:category | (Smalltalk organization classesInCategory: category) do: [:e | self quickFillWithClass: e]] >> Smalltalk globals organization no senders? Remove? Class categories are an obsolete concepts so we should clean this. NOCompletionTable comment I keep track of symbols to be retrieved in ocompletion algorithm. Unlike ecompletion, I learn something and try to retrieve users last occurences of typing sequences first, then the system symbols which match. TODO remove stupid symbols (like packages and method extensions and timestamp) > Hello to all, > > I am a student at the Faculty of Information Technology of the Czech > Technical University in Prague and I decided to make Code Completion in > Pharo the topic of my Bachelor's degree and try to >improve it. > > If you have suggestions on what should change / how to change it so you > would be satisfied with it, I will appreciate any input on this topic. > It would mean a lot to me if you could find two minutes >and take this > survey (there are only like 7 questions, so it won't take longer than > those 2 minutes): > > https://docs.google.com/forms/d/e/1FAIpQLSd7lx6gIN079-vMvMwRsnA_AX_e_7sZEz2… > > Regards > > > Lukas -- Using Opera's mail client: http://www.opera.com/mail/

Feb. 17, 2017

Re: [Pharo-dev] debugger evaluator tab for temporary variables

by Denis Kudriashov

2017-02-17 10:36 GMT+01:00 Ben Coman <btc(a)openinworld.com>: > Seems like you misread my idea (3am syndrome again?) > You wouldn't need to select "self". > The idea is in the first (left hand) pane you would flip from the > [Variables] to an [Evaluator] tab > this hiding all variables but having access to all the same ones from > the [Evaluator] tab. > Ok, I see :)

Feb. 17, 2017

Re: [Pharo-dev] debugger evaluator tab for temporary variables

by Ben Coman

On Fri, Feb 17, 2017 at 4:55 PM, Denis Kudriashov <dionisiydk(a)gmail.com> wrote: > > 2017-02-17 3:01 GMT+01:00 Ben Coman <btc(a)openinworld.com>: >> >> A minor irritation with the debugger is that the [Evaluator] tab is >> only for the object of the selected instance variable. > > > Evaluator appears only when you select something in table. It could be any > object which can have inst vars with same names as temps. > Do you wan't this behaviour when "self" is selected? > In that case newbies can ask why temps are not available when other objects > are selected. Seems like you misread my idea (3am syndrome again?) You wouldn't need to select "self". The idea is in the first (left hand) pane you would flip from the [Variables] to an [Evaluator] tab this hiding all variables but having access to all the same ones from the [Evaluator] tab. On Fri, Feb 17, 2017 at 3:58 PM, Tudor Girba <tudor(a)tudorgirba.com> wrote: > Please open an issue for this and add Andrei and I to it. https://pharo.fogbugz.com/f/cases/19720/Add-method-scoped-Evaluator-tab-to-… cheers -ben

Feb. 17, 2017