January 2023 - Pharo-dev - lists.pharo.org

Sockets not working under heavy VM load?
by Pavel Krivanek Oct. 3, 2023

Oct. 3, 2023

Hi, we have some performance issues with Glorp over P3 and, after some investigation, I have a theory that could explain it. And I would like to know your opinion if it is correct. To reproduce this issue, you can try something easy. Just create a process with normal or higher priority that will try to read something from a socket. And, in parallel, do in a separate thread an operation that takes all the processor time for several seconds. ```` [ 5 seconds wait. Time now traceCr. ZnClient new url: 'https://www.google.com'; get. Time now traceCr. ] forkAt: 60. [1000 timesRepeat: [ 10000 factorial ] ] timeToRun ```` (you may try a different safe server address) The processes run in parallel until the time when the first process tries to obtain data from the socket. Then it hangs and waits until finishing of the second process. My theory is that the VM socket plugin is able to receive incoming data from the connection and save it into the buffer. In the meantime, the reading process is waiting on the socket semaphore. But it looks like the plugin is not able to release the semaphore while the VM is actually doing something. It needs to wait until the VM has nothing useful to do, and then it is finally able to release the semaphores so Pharo processes are resumed and can start to read. Is my theory right? Cheers, -- Pavel

2 2

Code quality: tagging unsent selectors (e.g. in tests)
by Marcus Denker Jan. 30, 2023

Jan. 30, 2023

Hi, If you have in your code a message send with a selector that is not implemented anywhere, it likely will result in a DNU - but not if you compile a method on the fly (used in tests often) - if the DNU is catched (e.g. for a proxy pattern) - if the code is compatibility code, never executed in this version of the system We can ban the rule, but that just stopps checking for the whole method for all selectors, what we want is to communicate that *this selector* is ok in this special use (per method granularity is ok). SystemNaviation>>#allUnimplementedCalls did have the feature that one can tag selectors that should be ignored. This uses the pragma #ignoreUnimplementedCalls: see Context class >> initializePrimitiveSimulators <ignoreUnimplementedCalls: #(#registerPrimitiveSimulators)> This was used in just one method, I think because a) the rule did not take it into account and b) the terminology is just wrong (âCallâ). I think - ReSentNotImplementedRule should takes this pragma into account - change the bad terminology âallUnimplementedCalls" - use pragma: <ignoreNotImplementedSelectors: > PR is here: https://github.com/pharo-project/pharo/pull/12295 (what the PR does not yet do is to change syntax highlight to not color the selector red, this could be done later, too)

3 5

This week (4/2023) on the Pharo Issue Tracker
by Marcus Denker Jan. 27, 2023

Jan. 27, 2023

This week, we merged 28 Pull requests, fixing 31 issues Fixes ===== - Update BaselineOfPharo with Roassal3 version v0.9.9L #12430 https://github.com/pharo-project/pharo/pull/12430 - checkIfMustRefresh in OmBlockFileStore>>blockForLocalNameAsInteger: #12197 https://github.com/pharo-project/pharo/pull/12197 - Fix for multiple Epicea issues #12375 https://github.com/pharo-project/pharo/pull/12375 - FIX CI issues - FFIExternalString>>#basicHandle:at: calls deprecated message #12405 https://github.com/pharo-project/pharo/pull/12405 - Fix for "Browse this version in Calypso" Error after clicking an unloaded package #12423 https://github.com/pharo-project/pharo/pull/12423 - 12410-Minimal-Image-does-not-open-ChangesFile-correctly #12411 https://github.com/pharo-project/pharo/pull/12411 - fix testTallyInstructions #12393 https://github.com/pharo-project/pharo/pull/12393 - Spotter is broken> DNU #tools #12397 https://github.com/pharo-project/pharo/pull/12397 Cleanups ======== - RubScrolledTextMorph>>#mouseWheel: sends unimplemented message #scrolâ¦ #12402 https://github.com/pharo-project/pharo/pull/12402 - 12416-Fixing-Classes-Reported-by-NoUnusedVariablesLeftTest #12417 https://github.com/pharo-project/pharo/pull/12417 - 12413-Improving-the-dependency-of-Formatter-Settings-with-EFFormatter #12414 https://github.com/pharo-project/pharo/pull/12414 - Move #asNumber from AST-Core to Kernel (where it is also already tested) #12388 https://github.com/pharo-project/pharo/pull/12388 - extract return from ifTrue:ifFalse: in Integer>>#printStringRadix: #12433 https://github.com/pharo-project/pharo/pull/12433 - 11x Cleanups of the kind Cleanup: Condense Calypso packages #12284 https://github.com/pharo-project/pharo/pull/12284 this Changelog is archived at https://github.com/pharo-project/pharo-changelogs/blob/master/weekly/2023/4…

1 0

[Issue Tracker] reviews needed
by Marcus Denker Jan. 26, 2023

Jan. 26, 2023

Hi, I wanted again to make some noise about the issue tracker. Yes, we merge 20-40 PRs per week, but even with that, there are many PRs in the queueâ¦ e.g. - Complete symbols #11918 https://github.com/pharo-project/pharo/pull/11918 - 12128-Poor-performance-with-large-WriteStreams #12205 https://github.com/pharo-project/pharo/pull/12205 - Fix Calypso browsing unloaded package artifacts from Iceberg #12425 https://github.com/pharo-project/pharo/pull/12425 - Introducing a Color Palette to cache the allocation of Color objects by tht UITheme #12419 https://github.com/pharo-project/pharo/pull/12419 We have open 35 PRs, the complete list is here: https://github.com/pharo-project/pharo/pulls some random notes: - reviews are always helpful, and even if there is already a positive review, a second one helps. - one part of a review is to check that the code is not doing obvious bad things (like un-doing older changes, dumb mistakesâ¦.) if you feel unsure what you can do as you do not really understand the change, that one you can check (and just say that in the review). - if you do PRs e.g with a pair, ask the person that did not commit to add a real review so it is clear that there where already two eyes on the code - if you work in a team, ask your team mates to review your code especially if you use that fix already internally - if you use the fix in your project already, add a review to just state that - if you see a fix where you think âI hope this will be merged soonâ, if you review, you will make it much easierâ¦ - if you see a fix that is reviewed, green and not merged and you do not understand why: do a review - consider just reviewing / doing *something* once every week. or even once every day, instead of letting it all pile up. Marcus

1 0

Pharo11 32 bits images missing?
by Pierre Misse Chanabier Jan. 23, 2023

Jan. 23, 2023

Hello, Apparently Pharo 11 does not build 32 bit images [1]. Did I miss an announcement? Pierre http://files.pharo.org/image/110/

2 1

[ANN] Pharo Mastodon
by Marcus Denker Jan. 20, 2023

Jan. 20, 2023

Yes, not really news: Since Jan 4, 2018 we have a mastodon account. Esteban created that very early on, for now it re-shares the @pharoproject twitter account. https://mastodon.social/@pharoproject <https://mastodon.social/@pharoproject> I have finally updated https://www.pharo.org/community to link to it. Marcus

1 0

This week (3/2023) on the Pharo Issue Tracker
by Marcus Denker Jan. 20, 2023

Jan. 20, 2023

Features ======== - Add ifCutOn:doWithCutAndUncuts: #12374 https://github.com/pharo-project/pharo/pull/12374 - Lint rule warning for Calypso to warn unconditional recursion #12389 https://github.com/pharo-project/pharo/pull/12389 - Add #normalizeCamelCase and test #12373 https://github.com/pharo-project/pharo/pull/12373 - Add Generator class>>onDo: #12395 https://github.com/pharo-project/pharo/pull/12395 Fixes ===== - Variables: Fix Prrinting and Fluid Class definition that uses ClassVariable #12333 https://github.com/pharo-project/pharo/pull/12333 - Spotter is broken> DNU #tools #12397 https://github.com/pharo-project/pharo/pull/12397 - fixes #1659 IceCredentialStore did not understand #repositoryModel #1661 https://github.com/pharo-vcs/iceberg/pull/1661 - Issue 1275: Notify scroll changes #1344 https://github.com/pharo-spec/Spec/pull/1344 - fix tags access #1660 https://github.com/pharo-vcs/iceberg/pull/1660 Cleanup ======= - Move SmallInteger class>>#guideToDivision comment to class comment. #12391 https://github.com/pharo-project/pharo/pull/12391 - avoid hardcoding smalltalk everywhere and just factor out some use foâ¦ #449 https://github.com/pharo-spec/NewTools/pull/449 - Refactoring/categorizing unexposed refactorings #12370 https://github.com/pharo-project/pharo/pull/12370 - 20x cleanups of the kind Cleanup: Condense FFI-Kernel package #12328 https://github.com/pharo-project/pharo/pull/12328 - extract return from ifTrue:ifFalse: expression and whitespaces at the eol #12332 https://github.com/pharo-project/pharo/pull/12332 - extract return from ifTrue:ifFalse: expression #12177 https://github.com/pharo-project/pharo/pull/12177

1 0

Fwd: Iceberg in github action
by Pierre Misse Chanabier Jan. 14, 2023

Jan. 14, 2023

Hi, After quite a bit of work and investigating, we are finally able to use iceberg scripting in smalltalkci :) https://github.com/hpi-swa/smalltalkCI/pull/582 I'm using it in my Polyphemus repository if you want an example (https://github.com/hogoww/Polyphemus/actions) Cheers, Pierre

1 0

This week (2/2023) on the Pharo Issue Tracker
by Marcus Denker Jan. 13, 2023

Jan. 13, 2023

42 PRs merged, 40 Issues closed Features ======= - RubTextEditor/TextEditor: Make ctrl+backspace delete the previous word #12096 https://github.com/pharo-project/pharo/pull/12096 - Add source and target labels to SpChooserPresenter ( #1341) #1342 https://github.com/pharo-spec/Spec/pull/1342 Git/Iceberg ============ - add a manual way to add/remove tags #1657 https://github.com/pharo-vcs/iceberg/pull/1657 - add default tag "system" to bootstrap registered projects #12303 https://github.com/pharo-project/pharo/pull/12303 - fix bad categorizations #1655 https://github.com/pharo-vcs/iceberg/pull/1655 - Adapt to latest spec2 pass2 #1632 https://github.com/pharo-vcs/iceberg/pull/1632 Refactoring Engine ================== - Cleaning RB issues #12151 https://github.com/pharo-project/pharo/pull/12151 - Refactorings cleanup #12256 https://github.com/pharo-project/pharo/pull/12256 Fixes ===== - Fix refactoring using old class def #12195 https://github.com/pharo-project/pharo/pull/12195 - remove ByteLayout from FFIBackend and children #12225 https://github.com/pharo-project/pharo/pull/12225 - 12289-Refactoring-Tests-are-failling-when-using-Fluid-syntax #12290 https://github.com/pharo-project/pharo/pull/12290 - Fix hasSelector:specialSelectorIndex: on FFI methods that are compiled #12305 https://github.com/pharo-project/pharo/pull/12305 - 12300-Compiler-is-slow-with-string-literals-containing-exclamation-marks #12306 https://github.com/pharo-project/pharo/pull/12306 Cleanups ======== - 18 whitespace cleanups like Cleanup: Condense CodeImport packages #12229 https://github.com/pharo-project/pharo/pull/12229 - 5 remove return out of isTrue:ifFalse: lke extract return from ifTrue:ifFalse: expression in Number floor #12292 https://github.com/pharo-project/pharo/pull/12292 - remove #Numeric from source code. #12265 https://github.com/pharo-project/pharo/pull/12265 - 12307 remove gt pragma #12308 https://github.com/pharo-project/pharo/pull/12308 - Unncessary class RBMissingOpenerErrorNode #12313 https://github.com/pharo-project/pharo/pull/12313 - Class RBShortAssignmentToken is not needed anymore #12315 https://github.com/pharo-project/pharo/pull/12315 Comments ======== - Improving layoutFrame comments and add examples (fixes #12259) #12260 https://github.com/pharo-project/pharo/pull/12260 - extract comments as runnable example in Number>>#// #12263 https://github.com/pharo-project/pharo/pull/12263 - use isInstanceSide message in example #12270 https://github.com/pharo-project/pharo/pull/12270

1 0

[Website] some fixes: links to mailinglist and newsletter
by Marcus Denker Jan. 12, 2023

Jan. 12, 2023

Hi, A small update to https://www.pharo.org/community I fixed the links to the mailinglist and the newsletter subcription. (there are real âaboutâ pages.. I guess I was not the only one complaining that this is neededâ¦) https://lists.pharo.org/list/pharo-dev.lists.pharo.org https://lists.pharo.org/list/pharo-users.lists.pharo.org <https://lists.pharo.org/list/pharo-users.lists.pharo.org> (this closes https://github.com/pharo-project/pharo-site/issues/13) Marcus

1 0