Squeak-dev May 2020

squeak-dev@lists.squeakfoundation.org

43 participants
265 discussions

Fixed: squeak-smalltalk/squeak-app#1679 (squeak-trunk - 4d0475b)
by Travis CI 06 May '20

06 May '20

Build Update for squeak-smalltalk/squeak-app ------------------------------------- Build: #1679 Status: Fixed Duration: 20 mins and 58 secs Commit: 4d0475b (squeak-trunk) Author: Fabio Niephaus Message: Silence security setup code View the changeset: https://github.com/squeak-smalltalk/squeak-app/compare/2c5632597eb4...4d047… View the full build log and details: https://travis-ci.org/github/squeak-smalltalk/squeak-app/builds/683807138?u… -- You can unsubscribe from build emails from the squeak-smalltalk/squeak-app repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=8901856&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

Still Failing: squeak-smalltalk/squeak-app#1678 (squeak-trunk - 2c56325)
by Travis CI 06 May '20

06 May '20

Build Update for squeak-smalltalk/squeak-app ------------------------------------- Build: #1678 Status: Still Failing Duration: 17 mins and 59 secs Commit: 2c56325 (squeak-trunk) Author: Fabio Niephaus Message: Try installing xcnotary without brew View the changeset: https://github.com/squeak-smalltalk/squeak-app/compare/a193e5d2f214...2c563… View the full build log and details: https://travis-ci.org/github/squeak-smalltalk/squeak-app/builds/683798558?u… -- You can unsubscribe from build emails from the squeak-smalltalk/squeak-app repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=8901856&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

Still Failing: squeak-smalltalk/squeak-app#1677 (squeak-trunk - a193e5d)
by Travis CI 06 May '20

06 May '20

Build Update for squeak-smalltalk/squeak-app ------------------------------------- Build: #1677 Status: Still Failing Duration: 23 mins and 57 secs Commit: a193e5d (squeak-trunk) Author: Fabio Niephaus Message: Use xcode11.4 View the changeset: https://github.com/squeak-smalltalk/squeak-app/compare/7a35b044a5d4...a193e… View the full build log and details: https://travis-ci.org/github/squeak-smalltalk/squeak-app/builds/683787681?u… -- You can unsubscribe from build emails from the squeak-smalltalk/squeak-app repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=8901856&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

Still Failing: squeak-smalltalk/squeak-app#1676 (squeak-trunk - 7a35b04)
by Travis CI 06 May '20

06 May '20

Build Update for squeak-smalltalk/squeak-app ------------------------------------- Build: #1676 Status: Still Failing Duration: 15 mins and 3 secs Commit: 7a35b04 (squeak-trunk) Author: Fabio Niephaus Message: Install xcnotary from Catalina bottle and disable automatic brew cleanup and update again View the changeset: https://github.com/squeak-smalltalk/squeak-app/compare/6e78e6125b3b...7a35b… View the full build log and details: https://travis-ci.org/github/squeak-smalltalk/squeak-app/builds/683737112?u… -- You can unsubscribe from build emails from the squeak-smalltalk/squeak-app repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=8901856&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

Still Failing: squeak-smalltalk/squeak-app#1675 (squeak-trunk - 6e78e61)
by Travis CI 06 May '20

06 May '20

Build Update for squeak-smalltalk/squeak-app ------------------------------------- Build: #1675 Status: Still Failing Duration: 24 mins and 19 secs Commit: 6e78e61 (squeak-trunk) Author: Fabio Niephaus Message: Use xcode10.1 View the changeset: https://github.com/squeak-smalltalk/squeak-app/compare/cc8405a4e3e6...6e78e… View the full build log and details: https://travis-ci.org/github/squeak-smalltalk/squeak-app/builds/683725282?u… -- You can unsubscribe from build emails from the squeak-smalltalk/squeak-app repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=8901856&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

Still Failing: squeak-smalltalk/squeak-app#1674 (squeak-trunk - cc8405a)
by Travis CI 06 May '20

06 May '20

Build Update for squeak-smalltalk/squeak-app ------------------------------------- Build: #1674 Status: Still Failing Duration: 15 mins and 1 sec Commit: cc8405a (squeak-trunk) Author: Fabio Niephaus Message: Use xcode10.1 and avoid brew's auto updates View the changeset: https://github.com/squeak-smalltalk/squeak-app/compare/94ddf3051445...cc840… View the full build log and details: https://travis-ci.org/github/squeak-smalltalk/squeak-app/builds/683719172?u… -- You can unsubscribe from build emails from the squeak-smalltalk/squeak-app repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=8901856&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

The Trunk: Kernel-eem.1325.mcz
by commits＠source.squeak.org 06 May '20

06 May '20

Eliot Miranda uploaded a new version of Kernel to project The Trunk: http://source.squeak.org/trunk/Kernel-eem.1325.mcz ==================== Summary ==================== Name: Kernel-eem.1325 Author: eem Time: 1 May 2020, 10:38:26.143286 am UUID: fdfc5299-b643-4879-a7ca-716d825f6988 Ancestors: Kernel-eem.1324 Minor clean-up from the last commit. We don't really need compiledBlock[:], so do without. =============== Diff against Kernel-eem.1324 =============== Item was changed: ----- Method: FullBlockClosure>>= (in category 'comparing') ----- = aClosure self == aClosure ifTrue: [^true]. aClosure class == self class ifFalse: [^false]. + ^startpcOrMethod = aClosure method + and: [outerContext = aClosure outerContext or: [self isClean]]! - startpcOrMethod = aClosure compiledBlock ifFalse: [^false]. - ^outerContext = aClosure outerContext or: [self isClean]! Item was removed: - ----- Method: FullBlockClosure>>compiledBlock (in category 'accessing') ----- - compiledBlock - "To be able to inherit from BlockClosure" - ^ startpcOrMethod! Item was removed: - ----- Method: FullBlockClosure>>compiledBlock: (in category 'accessing') ----- - compiledBlock: aCompiledBlock - "To be able to inherit from BlockClosure" - startpcOrMethod := aCompiledBlock!

3 5

Broken: squeak-smalltalk/squeak-app#1673 (squeak-trunk - 94ddf30)
by Travis CI 06 May '20

06 May '20

Build Update for squeak-smalltalk/squeak-app ------------------------------------- Build: #1673 Status: Broken Duration: 26 mins and 1 sec Commit: 94ddf30 (squeak-trunk) Author: Fabio Niephaus Message: `brew update` not needed for installing xcnotary View the changeset: https://github.com/squeak-smalltalk/squeak-app/compare/db0dd0b58035...94ddf… View the full build log and details: https://travis-ci.org/github/squeak-smalltalk/squeak-app/builds/683708424?u… -- You can unsubscribe from build emails from the squeak-smalltalk/squeak-app repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=8901856&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

Re: [squeak-dev] [Pharo-users] mentor question 4
by Trygve Reenskaug 05 May '20

05 May '20

Unfortunately, I have lost the original problem specification, so I have answered a different question. Could you please help me by repeating the original problem specification in full? I agree that my solution is not 'obviously correct' since it doesn't answer the right question. In one version of the solution, I had a prologue that catered for spaces, etc. Not having a textual form of the algorithm that provided for the extensions, I chose to omit them. On tirsdag.05.05.2020 15:20, Richard O'Keefe wrote: > The irony is that the code I was responding to ISN'T obviously correct. > Indeed, I found it rather puzzling. > The problem specification says that the input string may contain digits > AND SPACES. The original message includes this: > > Strings of length 1 or less are not valid. Spaces are allowed in the > input, but they should be stripped before checking. All other > non-digit characters are disallowed. > > Now it isn't clear what "disallowed" means. I took it to mean "may occur and > should simply mean the input is rejected as invalid." Perhaps "may not occur" > was the intention. So we shall not quibble about such characters. > > But I can't for the life of me figure out how Trygve's code checks for spaces. > One reason this is an issue is that the behaviour of #digitValue is not > consistent between systems. > Character space digitValue > does not exist in the ANSI standard > answers -1 in many Smalltalks (which is a pain) > answers a positive integer that can't be mistake for a digit in my Smalltalk > raises an exception in some Smalltalks. > > This is a comment I now have in my Smalltalk library for #digitValue > "This is in the Blue Book, but unspecified on non-digits. > Squeak, Pharo, Dolphin, VW, VAST, and Apple Smalltalk-80 > answer -1 for characters that are not digits (or ASCII letters), > which is unfortunate but consistent with Inside Smalltalk > which specifies this result for non-digits. > ST/X and GST raise an exception which is worse. > Digitalk ST/V documentation doesn't specify the result. > This selector is *much* easier to use safely if it > returns a 'large' (>= 36) value for non-digits." > > Let's compare three versions, the two I compared last time, > and the "version A" code I discussed before, which to my mind > is fairly readable. > > "Don't add slowness": 1 (normalised time) > "Trygve's code": 6.5 > "High level code": 30.6 (or 4.7 times slower than Trygve's) > > Here's the "High level code". > ^(aString allSatisfy: [:each | each isSpace or: [each isDigit]]) and: [ > |digitsReverse| > digitsReverse := (aString select: [:each | each isDigit]) reverse. > digitsReverse size > 1 and: [ > |evens odds evenSum oddSum| > odds := digitsReverse withIndexSelect: [:y :i | i odd]. > evens := digitsReverse withIndexSelect: [:x :i | i even]. > oddSum := odds detectSum: [:y | y digitValue]. > evenSum := evens detectSum: [:x | > #(0 2 4 6 8 1 3 5 7 9) at: x digitValue + 1]. > (oddSum + evenSum) \\ 10 = 0]] > > This is the kind of code I was recommending that Roelof write. > > As a rough guide, by counting traversals (including ones inside existing > methods), I'd expect the "high level" code to be at least 10 times slower > than the "no added slowness" code. > > We are in vehement agreement that there is a time to write high level > really obvious easily testable and debuggable code, and that's most > of the time, especially with programming exercises. > > I hope that we are also in agreement that factors of 30 (or even 6) > *can* be a serious problem. I mean, if I wanted something that slow, > I'd use Ruby. > > I hope we are also agreed that (with the exception of investigations > like this one) the time to hack on something to make it faster is AFTER > you have profiled it and determined that you have a problem. > > But I respectfully suggest that there is a difference taking slowness OUT > and simply not going out of your way to add slowness in the first place. > > I'd also like to remark that my preference for methods that traverse a > sequence exactly once has more to do with Smalltalk protocols than > with efficiency. If the only method I perform on an object is #do: > the method will work just as well for readable streams as for > collections. If the only method I perform on an object is #reverseDo: > the method will work just as well for Read[Write]Streams as for > SequenceReadableCollections, at least in my library. It's just like > trying to write #mean so that it works for Durations as well as Numbers. > > Oh heck, I suppose I should point out that much of the overheads in > this case could be eliminated by a Self-style compiler doing dynamic > inlining + loop fusion. There's no reason *in principle*, given enough > people, money, and time, that the differences couldn't be greatly > reduced in Pharo. > > On Tue, 5 May 2020 at 21:50, Trygve Reenskaug <trygver(a)ifi.uio.no> wrote: >> Richard, >> >> Thank you for looking at the code. It is comforting to learn that the code has been executed for a large number of examples without breaking. The code is not primarily written for execution but for being read and checked by the human end user. It would be nice if we could also check that it gave the right answers, but I don't know how to do that. >> >> The first question is: Can a human domain expert read the code and sign their name for its correctness? >> >> >> When this is achieved, a programming expert will transcribe the first code to a professional quality program. This time, the second code should be reviewed by an independent programmer who signs their name for its correct transcription from the first version. >> >> --Trygve >> >> PS: In his 1991 Turing Award Lecture, Tony Hoare said: "There are two ways of constructing a software design: One way is to make it so simple that there are obviously no deficiencies and the other is to make it so complicated that there are no obvious deficiencies. The first method is far more difficult." >> >> --Trygve >> >> On tirsdag.05.05.2020 04:41, Richard O'Keefe wrote: >> >> As a coding experiment, I adapted Trygve Reenskoug's code to my >> Smalltalk compiler, put in my code slightly tweaked, and benchmarked >> them on randomly generated data. >> >> Result: a factor of 6.3. >> >> In Squeak it was a factor of ten. >> >> I had not, in all honesty, expected it to to be so high. >> >> On Tue, 5 May 2020 at 02:00, Trygve Reenskaug <trygver(a)ifi.uio.no> wrote: >> >> A coding experiment. >> Consider a Scrum development environment. Every programming team has an end user as a member. >> The team's task is to code a credit card validity check. >> A first goal is that the user representative shall read the code and agree that it is a correct rendering of their code checker: >> >> luhnTest: trialNumber >> | s1 odd s2 even charValue reverse | >> ----------------------------------------------- >> " Luhn test according to Rosetta" >> "Reverse the order of the digits in the number." >> reverse := trialNumber reversed. >> "Take the first, third, ... and every other odd digit in the reversed digits and sum them to form the partial sum s1" >> s1 := 0. >> odd := true. >> reverse do: >> [:char | >> odd >> ifTrue: [ >> s1 := s1 + char digitValue. >> ]. >> odd := odd not >> ]. >> "Taking the second, fourth ... and every other even digit in the reversed digits: >> Multiply each digit by two and sum the digits if the answer is greater than nine to form partial sums for the even digits" >> "The subtracting 9 gives the same answer. " >> "Sum the partial sums of the even digits to form s2" >> s2 := 0. >> even := false. >> reverse do: >> [:char | >> even >> ifTrue: [ >> charValue := char digitValue * 2. >> charValue > 9 ifTrue: [charValue := charValue - 9]. >> s2 := s2 + charValue >> ]. >> even := even not >> ]. >> "If s1 + s2 ends in zero then the original number is in the form of a valid credit card number as verified by the Luhn test." >> ^(s1 + s2) asString last = $0 >> --------------------------------- >> Once this step is completed, the next step will be to make the code right without altering the algorithm (refactoring). The result should be readable and follow the team's conventions. >> >> >> P.S. code attached. >> >> >> -- >> >> The essence of object orientation is that objects collaborate to achieve a goal. >> Trygve Reenskaug mailto: trygver(a)ifi.uio.no >> Morgedalsvn. 5A http://folk.uio.no/trygver/ >> N-0378 Oslo http://fullOO.info >> Norway Tel: (+47) 468 58 625 -- /The essence of object orientation is that objects collaborateto achieve a goal. / Trygve Reenskaug mailto: trygver(a)ifi.uio.no <mailto:%20trygver@ifi.uio.no> Morgedalsvn. 5A http://folk.uio.no/trygver/ N-0378 Oslo http://fullOO.info Norway Tel: (+47) 468 58 625

1 0

The Trunk: Help-Squeak-Project-nice.80.mcz
by commits＠source.squeak.org 05 May '20

05 May '20

Nicolas Cellier uploaded a new version of Help-Squeak-Project to project The Trunk: http://source.squeak.org/trunk/Help-Squeak-Project-nice.80.mcz ==================== Summary ==================== Name: Help-Squeak-Project-nice.80 Author: nice Time: 29 April 2020, 10:36:37.48125 pm UUID: e5ced11f-73d5-1d4f-a9ac-5af520cc1abe Ancestors: Help-Squeak-Project-mt.79 Let Help survive to Inspector refactorings. =============== Diff against Help-Squeak-Project-mt.79 =============== Item was changed: ----- Method: SqueakMessageCategoriesHelp>>menu (in category 'menu') ----- menu "This category includes all methods related to menus, often context menus in particular. It contains methods defining the menu as well as often callbacks for the labels of menu items or the actions to be executed. Sometimes the menus are split up in different categories if there are multiple menus or submenus." + ^ {Inspector>>#mainFieldListMenu: . SystemWindow>>#makeClosable}! - ^ {DictionaryInspector>>#mainFieldListMenu: . SystemWindow>>#makeClosable}!

3 3

Jump to page: