- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] ListModel announcer insanity

by phil＠highoctane.be

Le 15 dÃ©c. 2014 13:00, "Henrik Johansen" <henrik.s.johansen(a)veloxit.no> a Ã©crit : > > I thought I'd fix a small bug following https://pharo.fogbugz.com/f/cases/14604 , the DiffMorphChangeRecordConverters referenceList does not get updated to the new model's list when reverting a method, so after reverting, the left pane will always show (and compare against) the last version, rather than the previous from selected. > > No problem, as per the discussion of Spec's excessive annoucement wrapping/ValueHolder appetite, let's try do the simple thing and add a small Announcement subscription, essentially: > browser listModel announcer when: ValueChanged send: #updateReferences: to: aConverter. > > <rant> > To my surprise, that did not work. > Hmmm, must be a different announcement then, ListChanged maybe. After all, it makes sense that a MultiSelectionInList has multiple events that subscribers might be interested in. > > So I started looking at ListModel, <insert relevant explicits here>! > There are, count them and weep, *32* different NewValueModels instance variables, each with different announcers. > > In the case I was looking at, in addition to a listHolder inst var (a CollectionValueHolder), there's a listAnnouncer inst var (an Announcer), so just finding the one through which announcements are made is next to impossible (with the ListModel's own announcer, that's three different ones that would potentially make sense), at least without consulting the *offical* API; #whenListChanged: > > Turns out, in the current implementation, listAnnouncer is the one I want. > Which isn't exposed through an accessor, so you can't even use it to subscribe directly. Yay. > </rant> > > If someone has time to restructure this, I would like to suggest changing the internals radically; > - Introduce new Announcement types for all the different Value holders that make sense (SelectionChanged, ListChanged), and drop the rest. I mean, whenSortingBlockChanged: ?? Who on earth would be interested in that, and not already respond to ListChanged (or it's current equivalent) > - Reduce the amount of announcers to a saner number, say, 1 (or, in a pinch, 2). Yeah there are "Announcement storms" going on with Spec based tools. I think that it explains a lot of the sluggishness of some tools, like Nautilus. I saw these when doing some Nautilus plug-ins for my own use (e.g. bring back the original senders/implementors... buttons). Squeak tools feel very very snappy in comparison. Even on a superfast PC, Pharo as some annoying delays. Announcements are great but overusing them is receipe for serious head scratching and does not lend itself to easy discoverability of how things do work. Phil Phil > > Cheers, > Henry

Dec. 15, 2014

Re: [Pharo-dev] jenkins and static analysis

by Christophe Demarey

Le 15 dÃ©c. 2014 Ã 12:17, Norbert Hartl a Ã©crit : > >> Am 15.12.2014 um 12:05 schrieb Yuriy Tymchuk <yuriy.tymchuk(a)me.com>: >> >> Wellâ¦ there is this job: https://ci.inria.fr/pharo/view/3.0-Analysis/job/Pharo-3.0-CodeCritics/plot/… >> >> Itâs a bit complicated, but in Pharo4 rule checking got much easier, so you can script something buy yourself, or I can help you. The main question is what do you want to obtain? >> > Back then there was a mapping from lint warnings and errors to a checkstyle format file which can be feeded into the checkstyle plugin on jenkins. That would be fine. > I wouldn't be sad if the categories of lint rules could be seen in the jenkins graph. That might be possible to do but that is hard to configure on the job. The missing thing is an output for the Jenkins Violations plugin (https://wiki.jenkins-ci.org/display/JENKINS/Violations) We could have our own type (critics) or just re-use one (ex: pmd). I also wanted to do that but did not get the time yet. I also want that critics are checked in the same time as tests in jenkins jobs.

Dec. 15, 2014

ListModel announcer insanity

by Henrik Johansen

I thought I'd fix a small bug following https://pharo.fogbugz.com/f/cases/14604 , the DiffMorphChangeRecordConverters referenceList does not get updated to the new model's list when reverting a method, so after reverting, the left pane will always show (and compare against) the last version, rather than the previous from selected. No problem, as per the discussion of Spec's excessive annoucement wrapping/ValueHolder appetite, let's try do the simple thing and add a small Announcement subscription, essentially: browser listModel announcer when: ValueChanged send: #updateReferences: to: aConverter. <rant> To my surprise, that did not work. Hmmm, must be a different announcement then, ListChanged maybe. After all, it makes sense that a MultiSelectionInList has multiple events that subscribers might be interested in. So I started looking at ListModel, <insert relevant explicits here>! There are, count them and weep, *32* different NewValueModels instance variables, each with different announcers. In the case I was looking at, in addition to a listHolder inst var (a CollectionValueHolder), there's a listAnnouncer inst var (an Announcer), so just finding the one through which announcements are made is next to impossible (with the ListModel's own announcer, that's three different ones that would potentially make sense), at least without consulting the *offical* API; #whenListChanged: Turns out, in the current implementation, listAnnouncer is the one I want. Which isn't exposed through an accessor, so you can't even use it to subscribe directly. Yay. </rant> If someone has time to restructure this, I would like to suggest changing the internals radically; - Introduce new Announcement types for all the different Value holders that make sense (SelectionChanged, ListChanged), and drop the rest. I mean, whenSortingBlockChanged: ?? Who on earth would be interested in that, and not already respond to ListChanged (or it's current equivalent) - Reduce the amount of announcers to a saner number, say, 1 (or, in a pinch, 2). Cheers, Henry

Dec. 15, 2014

Re: [Pharo-dev] [Pharo-users] About Pharo books

by phil＠highoctane.be

We could do a git repo with all books as sumbodules and huge pillar.conf to build the Pharo encyclopedia... Le 15 dÃ©c. 2014 12:21, "kilon alios" <kilon.alios(a)gmail.com> a Ã©crit : > great work guys, thank you for your hard work. I would prefer it if I > could find all documentation in one place and I dont have to look into 10 > books to find what i want. Maybe have the books and one central place with > all documentation ? > > > On Mon, Dec 15, 2014 at 12:50 PM, Luc Fabresse <luc.fabresse(a)gmail.com> > wrote: >> >> Hi Guys, >> >> I've just pushed some commits >> in SquareBracketAssociates/PharoReadyForReviews >> and now it compiles. >> I will do it for the new repos: PharoInProgress and PharoLimbo soon. >> >> @Stef, now https://github.com/pillar-markup/book-skeleton to have the >> latest pillar compilation tool chain. >> >> Cheers, >> >> #Luc >> >> 2014-12-12 21:50 GMT+01:00 stepharo <stepharo(a)free.fr>: >>> >>> Hi guys >>> >>> I spent a couple of days cleaning PharoForTheEntreprise. So do not >>> stress if you chapters are not where they >>> were used to be :). >>> >>> Here is what I did now there are three stages for Chapters: >>> >>> - Limbo (some notes) >>> - WorkInProgress >>> - ReadyForReviews >>> >>> At the end of this flow we are thinking about the following books: >>> >>> - Entreprise Pharo: a Web Perspective >>> (Zinc, Neo.....) >>> - Spec >>> We decided to write a separate little book >>> >>> - Pharo: The Hidden Parts >>> The ideas is to have many little chapters on not common >>> aspects >>> >>> Stef >>> >>> PS: currently doing a pass on Smacc >>> >>>

Dec. 15, 2014

Re: [Pharo-dev] [Pharo-users] About Pharo books

by kilon alios

great work guys, thank you for your hard work. I would prefer it if I could find all documentation in one place and I dont have to look into 10 books to find what i want. Maybe have the books and one central place with all documentation ? On Mon, Dec 15, 2014 at 12:50 PM, Luc Fabresse <luc.fabresse(a)gmail.com> wrote: > > Hi Guys, > > I've just pushed some commits > in SquareBracketAssociates/PharoReadyForReviews > and now it compiles. > I will do it for the new repos: PharoInProgress and PharoLimbo soon. > > @Stef, now https://github.com/pillar-markup/book-skeleton to have the > latest pillar compilation tool chain. > > Cheers, > > #Luc > > 2014-12-12 21:50 GMT+01:00 stepharo <stepharo(a)free.fr>: >> >> Hi guys >> >> I spent a couple of days cleaning PharoForTheEntreprise. So do not stress >> if you chapters are not where they >> were used to be :). >> >> Here is what I did now there are three stages for Chapters: >> >> - Limbo (some notes) >> - WorkInProgress >> - ReadyForReviews >> >> At the end of this flow we are thinking about the following books: >> >> - Entreprise Pharo: a Web Perspective >> (Zinc, Neo.....) >> - Spec >> We decided to write a separate little book >> >> - Pharo: The Hidden Parts >> The ideas is to have many little chapters on not common >> aspects >> >> Stef >> >> PS: currently doing a pass on Smacc >> >>

Dec. 15, 2014

Re: [Pharo-dev] jenkins and static analysis

by Norbert Hartl

> Am 15.12.2014 um 12:05 schrieb Yuriy Tymchuk <yuriy.tymchuk(a)me.com>: > > Wellâ¦ there is this job: https://ci.inria.fr/pharo/view/3.0-Analysis/job/Pharo-3.0-CodeCritics/plot/… <https://ci.inria.fr/pharo/view/3.0-Analysis/job/Pharo-3.0-CodeCritics/plot/…> > > Itâs a bit complicated, but in Pharo4 rule checking got much easier, so you can script something buy yourself, or I can help you. The main question is what do you want to obtain? > Back then there was a mapping from lint warnings and errors to a checkstyle format file which can be feeded into the checkstyle plugin on jenkins. That would be fine. I wouldn't be sad if the categories of lint rules could be seen in the jenkins graph. That might be possible to do but that is hard to configure on the job. Norbert > Uko > >> On 15 Dec 2014, at 11:49, Norbert Hartl <norbert(a)hartl.name <mailto:norbert@hartl.name>> wrote: >> >> What are the best options at the moment to see the output of lint results in jenkins? Is there something to use with checkstyle or any other tools? >> I know the repo on >> >> http://smalltalkhub.com/mc/ClementBera/code-critic-jenkins/main <http://smalltalkhub.com/mc/ClementBera/code-critic-jenkins/main> >> >> but I cannot figure out what is the best way to use it. >> >> thanks, >> >> Norbert >> >

Dec. 15, 2014

Re: [Pharo-dev] jenkins and static analysis

by Yuriy Tymchuk

Wellâ¦ there is this job: https://ci.inria.fr/pharo/view/3.0-Analysis/job/Pharo-3.0-CodeCritics/plot/… <https://ci.inria.fr/pharo/view/3.0-Analysis/job/Pharo-3.0-CodeCritics/plot/…> Itâs a bit complicated, but in Pharo4 rule checking got much easier, so you can script something buy yourself, or I can help you. The main question is what do you want to obtain? Uko > On 15 Dec 2014, at 11:49, Norbert Hartl <norbert(a)hartl.name> wrote: > > What are the best options at the moment to see the output of lint results in jenkins? Is there something to use with checkstyle or any other tools? > I know the repo on > > http://smalltalkhub.com/mc/ClementBera/code-critic-jenkins/main <http://smalltalkhub.com/mc/ClementBera/code-critic-jenkins/main> > > but I cannot figure out what is the best way to use it. > > thanks, > > Norbert >

Dec. 15, 2014

Re: [Pharo-dev] [Pharo-users] About Pharo books

by Luc Fabresse

Hi Guys, I've just pushed some commits in SquareBracketAssociates/PharoReadyForReviews and now it compiles. I will do it for the new repos: PharoInProgress and PharoLimbo soon. @Stef, now https://github.com/pillar-markup/book-skeleton to have the latest pillar compilation tool chain. Cheers, #Luc 2014-12-12 21:50 GMT+01:00 stepharo <stepharo(a)free.fr>: > > Hi guys > > I spent a couple of days cleaning PharoForTheEntreprise. So do not stress > if you chapters are not where they > were used to be :). > > Here is what I did now there are three stages for Chapters: > > - Limbo (some notes) > - WorkInProgress > - ReadyForReviews > > At the end of this flow we are thinking about the following books: > > - Entreprise Pharo: a Web Perspective > (Zinc, Neo.....) > - Spec > We decided to write a separate little book > > - Pharo: The Hidden Parts > The ideas is to have many little chapters on not common aspects > > Stef > > PS: currently doing a pass on Smacc > >

Dec. 15, 2014

jenkins and static analysis

by Norbert Hartl

What are the best options at the moment to see the output of lint results in jenkins? Is there something to use with checkstyle or any other tools? I know the repo on http://smalltalkhub.com/mc/ClementBera/code-critic-jenkins/main <http://smalltalkhub.com/mc/ClementBera/code-critic-jenkins/main> but I cannot figure out what is the best way to use it. thanks, Norbert

Dec. 15, 2014

Re: [Pharo-dev] Funny mousedrag behavior over TxText-Morphs

by Igor Stasenko

The drag behavior over text was not inspected. This is feature this have not overlooked, so expect bumps. On 14 December 2014 at 21:14, stepharo <stepharo(a)free.fr> wrote: > I do not know exactly the status of the spec version. > I know that igor got many of problems managing the Morph into a text > because of circumvoluted morphic erosion of design. > I know that igor was not happy the way spec wat using text but I do not > know more than that. > > Stef > > -- Best regards, Igor Stasenko.

Dec. 15, 2014