O2: tester wanted!

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
17 messages Options
Reply | Threaded
Open this post in threaded view
|

O2: tester wanted!

Alexandre Bergel-4
Dear All,

For people who do not know, O2 is the browser David has implemented in  
the last few years. It replaces class-categories on the left-hand side  
pane with a list of packages. We are still away from having packages  
as real first class entities, but this is a good step toward. Note  
that some people find O2 a bit slower than other available code  
browsers.

David and I spent some time on fixing O2. O2 has now no dependencies  
to OB anymore.
We would like that people try it and submit bug reports on the google  
bug tracker. In a fresh 10505, you can load O2 by evaluating:

Gofer new squeaksource: 'MetacelloRepository'; package:  
'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:  
#loadDefault

O2 does not need OB whatsoever. O2 loads in a Core (I tried in a  
11165). Before and after loading O2, running OB tests give "548 run,  
547 passes, 1 failures" [*].

Cheers,
Alexandre

[*] the test have been loaded using (ConfigurationOfOmniBrowser  
project version: '1.1')  load: 'Dev Tests'
--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: O2: tester wanted!

Stéphane Ducasse
Thanks this is good.
And yes I did not make any progress on the package rewrite.
Doing too much pharo cleaning and project writing.
Stef

On Jan 19, 2010, at 11:03 PM, Alexandre Bergel wrote:

> Dear All,
>
> For people who do not know, O2 is the browser David has implemented in  
> the last few years. It replaces class-categories on the left-hand side  
> pane with a list of packages. We are still away from having packages  
> as real first class entities, but this is a good step toward. Note  
> that some people find O2 a bit slower than other available code  
> browsers.
>
> David and I spent some time on fixing O2. O2 has now no dependencies  
> to OB anymore.
> We would like that people try it and submit bug reports on the google  
> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>
> Gofer new squeaksource: 'MetacelloRepository'; package:  
> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:  
> #loadDefault
>
> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a  
> 11165). Before and after loading O2, running OB tests give "548 run,  
> 547 passes, 1 failures" [*].
>
> Cheers,
> Alexandre
>
> [*] the test have been loaded using (ConfigurationOfOmniBrowser  
> project version: '1.1')  load: 'Dev Tests'
> --
> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
> Alexandre Bergel  http://www.bergel.eu
> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>
>
>
>
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: O2: tester wanted!

Mariano Martinez Peck
In reply to this post by Alexandre Bergel-4


On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel <[hidden email]> wrote:
Dear All,

For people who do not know, O2 is the browser David has implemented in
the last few years. It replaces class-categories on the left-hand side
pane with a list of packages. We are still away from having packages
as real first class entities, but this is a good step toward. Note
that some people find O2 a bit slower than other available code
browsers.

David and I spent some time on fixing O2. O2 has now no dependencies
to OB anymore.
We would like that people try it and submit bug reports on the google
bug tracker. In a fresh 10505, you can load O2 by evaluating:

Gofer new squeaksource: 'MetacelloRepository'; package:
'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:
#loadDefault


Yes, but be aware that that piece of code load only the core, as you can see here:

        group: 'default' with: #('Core' );
            group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic' 'O2-Enhancements' );
            group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' );

   
As I remember, most of the problems of O2 were sometimes related to the integration to RB. So, if you really want to tests it, please, test it using the RB:

((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev'


As I already said to Alexandre and David in private, I will put O2 again in the next image. So, it would be cool if it is tested "completely".

 
O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
11165). Before and after loading O2, running OB tests give "548 run,
547 passes, 1 failures" [*].


Yes, if someone can help with that failure would be cool, I have already reported it.

 
Cheers,
Alexandre

[*] the test have been loaded using (ConfigurationOfOmniBrowser
project version: '1.1')  load: 'Dev Tests'
--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Adrian Lienhard
Thanks, Alex

I gave it a quick try. Installation into PharoCore 10505 worked and briefly clicking through the browser I haven't found any obvious problems.

However, there are 6 undeclared classes referenced in the code. Very likely when this code is run, O2 will fall over. Also 2 tests have errors and one fails.

HTH,
Adrian


On Jan 19, 2010, at 23:11 , Mariano Martinez Peck wrote:

> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel <
> [hidden email]> wrote:
>
>> Dear All,
>>
>> For people who do not know, O2 is the browser David has implemented in
>> the last few years. It replaces class-categories on the left-hand side
>> pane with a list of packages. We are still away from having packages
>> as real first class entities, but this is a good step toward. Note
>> that some people find O2 a bit slower than other available code
>> browsers.
>>
>> David and I spent some time on fixing O2. O2 has now no dependencies
>> to OB anymore.
>> We would like that people try it and submit bug reports on the google
>> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>>
>> Gofer new squeaksource: 'MetacelloRepository'; package:
>> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:
>> #loadDefault
>>
>>
> Yes, but be aware that that piece of code load only the core, as you can see
> here:
>
>        group: 'default' with: #('Core' );
>            group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic'
> 'O2-Enhancements' );
>            group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' );
>
>
> As I remember, most of the problems of O2 were sometimes related to the
> integration to RB. So, if you really want to tests it, please, test it using
> the RB:
>
> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev'
>
>
> As I already said to Alexandre and David in private, I will put O2 again in
> the next image. So, it would be cool if it is tested "completely".
>
>
>
>> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
>> 11165). Before and after loading O2, running OB tests give "548 run,
>> 547 passes, 1 failures" [*].
>>
>>
> Yes, if someone can help with that failure would be cool, I have already
> reported it.
>
>
>
>> Cheers,
>> Alexandre
>>
>> [*] the test have been loaded using (ConfigurationOfOmniBrowser
>> project version: '1.1')  load: 'Dev Tests'
>> --
>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>> Alexandre Bergel  http://www.bergel.eu
>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>
>>
>>
>>
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Igor Stasenko
In reply to this post by Alexandre Bergel-4
Is it virtually possible to run it in Squeak image, since its not
depends on OB anymore?

2010/1/20 Alexandre Bergel <[hidden email]>:

> Dear All,
>
> For people who do not know, O2 is the browser David has implemented in
> the last few years. It replaces class-categories on the left-hand side
> pane with a list of packages. We are still away from having packages
> as real first class entities, but this is a good step toward. Note
> that some people find O2 a bit slower than other available code
> browsers.
>
> David and I spent some time on fixing O2. O2 has now no dependencies
> to OB anymore.
> We would like that people try it and submit bug reports on the google
> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>
> Gofer new squeaksource: 'MetacelloRepository'; package:
> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:
> #loadDefault
>
> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
> 11165). Before and after loading O2, running OB tests give "548 run,
> 547 passes, 1 failures" [*].
>
> Cheers,
> Alexandre
>
> [*] the test have been loaded using (ConfigurationOfOmniBrowser
> project version: '1.1')  load: 'Dev Tests'
> --
> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
> Alexandre Bergel  http://www.bergel.eu
> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>
>
>
>
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>



--
Best regards,
Igor Stasenko AKA sig.

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Stefan Marr-4
In reply to this post by Mariano Martinez Peck
Hi:

Browser loaded like follows:

Gofer new squeaksource: 'MetacelloRepository'; package:
'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:
#loadDefault.

((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev'


In general, it works stable for me.
Have used it the whole afternoon.

Some small points I've found:

- selecting the 'all' protocol does not allow testing with cmd+t (and context menu)
  it reports that 0 tests have been executed

- the icons should have a transparent background, does not look nice when an item
  is selected

- code completion popups are not removed correctly, but I think that is not specific
  to O2
  Reproduce:
   1. type somewhere (code pane or search bar) to bring up completion
   2. then move the whole browser window
  That is only one of the few cases where such a popup is not removed.


Best
Stefan

On 19 Jan 2010, at 23:11, Mariano Martinez Peck wrote:

>
>
> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel <[hidden email]> wrote:
> Dear All,
>
> For people who do not know, O2 is the browser David has implemented in
> the last few years. It replaces class-categories on the left-hand side
> pane with a list of packages. We are still away from having packages
> as real first class entities, but this is a good step toward. Note
> that some people find O2 a bit slower than other available code
> browsers.
>
> David and I spent some time on fixing O2. O2 has now no dependencies
> to OB anymore.
> We would like that people try it and submit bug reports on the google
> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>
> Gofer new squeaksource: 'MetacelloRepository'; package:
> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:
> #loadDefault
>
>
> Yes, but be aware that that piece of code load only the core, as you can see here:
>
>         group: 'default' with: #('Core' );
>             group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic' 'O2-Enhancements' );
>             group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' );
>
>    
> As I remember, most of the problems of O2 were sometimes related to the integration to RB. So, if you really want to tests it, please, test it using the RB:
>
> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev'
>
>
> As I already said to Alexandre and David in private, I will put O2 again in the next image. So, it would be cool if it is tested "completely".
>
>  
> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
> 11165). Before and after loading O2, running OB tests give "548 run,
> 547 passes, 1 failures" [*].
>
>
> Yes, if someone can help with that failure would be cool, I have already reported it.
>
>  
> Cheers,
> Alexandre
>
> [*] the test have been loaded using (ConfigurationOfOmniBrowser
> project version: '1.1')  load: 'Dev Tests'
> --
> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
> Alexandre Bergel  http://www.bergel.eu
> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>
>
>
>
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project



--
Stefan Marr
Software Languages Lab
Vrije Universiteit Brussel
Pleinlaan 2 / B-1050 Brussels / Belgium
http://soft.vub.ac.be/~smarr
Phone: +32 2 629 3956
Fax:   +32 2 629 3525


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Mariano Martinez Peck


- code completion popups are not removed correctly, but I think that is not specific
 to O2
 Reproduce:
  1. type somewhere (code pane or search bar) to bring up completion
  2. then move the whole browser window
 That is only one of the few cases where such a popup is not removed.


you are a good tester :)   I checked and it also happens with OB. Can you open a bug ticket please?

If you identify other reproducible situations, please write them too

Thanks!
 


Best
Stefan

On 19 Jan 2010, at 23:11, Mariano Martinez Peck wrote:

>
>
> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel <[hidden email]> wrote:
> Dear All,
>
> For people who do not know, O2 is the browser David has implemented in
> the last few years. It replaces class-categories on the left-hand side
> pane with a list of packages. We are still away from having packages
> as real first class entities, but this is a good step toward. Note
> that some people find O2 a bit slower than other available code
> browsers.
>
> David and I spent some time on fixing O2. O2 has now no dependencies
> to OB anymore.
> We would like that people try it and submit bug reports on the google
> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>
> Gofer new squeaksource: 'MetacelloRepository'; package:
> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:
> #loadDefault
>
>
> Yes, but be aware that that piece of code load only the core, as you can see here:
>
>         group: 'default' with: #('Core' );
>             group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic' 'O2-Enhancements' );
>             group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' );
>
>
> As I remember, most of the problems of O2 were sometimes related to the integration to RB. So, if you really want to tests it, please, test it using the RB:
>
> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev'
>
>
> As I already said to Alexandre and David in private, I will put O2 again in the next image. So, it would be cool if it is tested "completely".
>
>
> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
> 11165). Before and after loading O2, running OB tests give "548 run,
> 547 passes, 1 failures" [*].
>
>
> Yes, if someone can help with that failure would be cool, I have already reported it.
>
>
> Cheers,
> Alexandre
>
> [*] the test have been loaded using (ConfigurationOfOmniBrowser
> project version: '1.1')  load: 'Dev Tests'
> --
> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
> Alexandre Bergel  http://www.bergel.eu
> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>
>
>
>
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project



--
Stefan Marr
Software Languages Lab
Vrije Universiteit Brussel
Pleinlaan 2 / B-1050 Brussels / Belgium
http://soft.vub.ac.be/~smarr
Phone: +32 2 629 3956
Fax:   +32 2 629 3525


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Alexandre Bergel
In reply to this post by Adrian Lienhard
> However, there are 6 undeclared classes referenced in the code. Very  
> likely when this code is run, O2 will fall over. Also 2 tests have  
> errors and one fails.

That cannot be left like that. I will seat down with David asap...

Alexandre

>
>
> On Jan 19, 2010, at 23:11 , Mariano Martinez Peck wrote:
>
>> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel <
>> [hidden email]> wrote:
>>
>>> Dear All,
>>>
>>> For people who do not know, O2 is the browser David has  
>>> implemented in
>>> the last few years. It replaces class-categories on the left-hand  
>>> side
>>> pane with a list of packages. We are still away from having packages
>>> as real first class entities, but this is a good step toward. Note
>>> that some people find O2 a bit slower than other available code
>>> browsers.
>>>
>>> David and I spent some time on fixing O2. O2 has now no dependencies
>>> to OB anymore.
>>> We would like that people try it and submit bug reports on the  
>>> google
>>> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>>>
>>> Gofer new squeaksource: 'MetacelloRepository'; package:
>>> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2)  
>>> perform:
>>> #loadDefault
>>>
>>>
>> Yes, but be aware that that piece of code load only the core, as  
>> you can see
>> here:
>>
>>       group: 'default' with: #('Core' );
>>           group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-
>> Morphic'
>> 'O2-Enhancements' );
>>           group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' );
>>
>>
>> As I remember, most of the problems of O2 were sometimes related to  
>> the
>> integration to RB. So, if you really want to tests it, please, test  
>> it using
>> the RB:
>>
>> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load:  
>> 'Dev'
>>
>>
>> As I already said to Alexandre and David in private, I will put O2  
>> again in
>> the next image. So, it would be cool if it is tested "completely".
>>
>>
>>
>>> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
>>> 11165). Before and after loading O2, running OB tests give "548 run,
>>> 547 passes, 1 failures" [*].
>>>
>>>
>> Yes, if someone can help with that failure would be cool, I have  
>> already
>> reported it.
>>
>>
>>
>>> Cheers,
>>> Alexandre
>>>
>>> [*] the test have been loaded using (ConfigurationOfOmniBrowser
>>> project version: '1.1')  load: 'Dev Tests'
>>> --
>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>>> Alexandre Bergel  http://www.bergel.eu
>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>>
>>>
>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Henrik Sperre Johansen
In reply to this post by Igor Stasenko
On 20.01.2010 15:59, Igor Stasenko wrote:
> Is it virtually possible to run it in Squeak image, since its not
> depends on OB anymore?
>    
Not quite.
1. In MCClassDefinition>> loadClass, isEmpty is sent to
traitComposition, since O2 includes classes with trait definitions as a
single trait, it'd have to be rewritten to:
composition := self traitComposition  ifNotNil:[: trait | Compiler
evaluate: trait].
     (composition isNil and:[class traitComposition isEmpty]) ifFalse:[
         class setTraitComposition: composition asTraitComposition.
     ].
in order to load them correctly.

2. In ConfigurationOfO2, the baseline10 would have to change spec for:
#pharo into spec for: #common.

After those two changes, it still won't work, as O2 relies on one of the
classes originally included (but really unrelated to) in the Traits
category, and this has been removed from Squeak.
If you make an external package with that, and modify the baseline to
include a spec for: #squeak which imports said package, it might.

Cheers,
Henry

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Henrik Sperre Johansen
In reply to this post by Alexandre Bergel-4
On 19.01.2010 23:03, Alexandre Bergel wrote:

> Dear All,
>
> For people who do not know, O2 is the browser David has implemented in
> the last few years. It replaces class-categories on the left-hand side
> pane with a list of packages. We are still away from having packages
> as real first class entities, but this is a good step toward. Note
> that some people find O2 a bit slower than other available code
> browsers.
>
> David and I spent some time on fixing O2. O2 has now no dependencies
> to OB anymore.
> We would like that people try it and submit bug reports on the google
> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>
> Gofer new squeaksource: 'MetacelloRepository'; package:
> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:
> #loadDefault
>
> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
> 11165). Before and after loading O2, running OB tests give "548 run,
> 547 passes, 1 failures" [*].
>
> Cheers,
> Alexandre
>
> [*] the test have been loaded using (ConfigurationOfOmniBrowser
> project version: '1.1')  load: 'Dev Tests'
>    

OR2PackageBrowser does not open, it lacks a defaultRootNode method
returning other than nil.
Is the focus mainly on the O2* browsers, or should OR2* also work?

Cheers,
Henry


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Alexandre Bergel
Well spotted. David?

Alexandre


On 20 Jan 2010, at 18:44, Henrik Sperre Johansen wrote:

> On 19.01.2010 23:03, Alexandre Bergel wrote:
>> Dear All,
>>
>> For people who do not know, O2 is the browser David has implemented  
>> in
>> the last few years. It replaces class-categories on the left-hand  
>> side
>> pane with a list of packages. We are still away from having packages
>> as real first class entities, but this is a good step toward. Note
>> that some people find O2 a bit slower than other available code
>> browsers.
>>
>> David and I spent some time on fixing O2. O2 has now no dependencies
>> to OB anymore.
>> We would like that people try it and submit bug reports on the google
>> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>>
>> Gofer new squeaksource: 'MetacelloRepository'; package:
>> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2)  
>> perform:
>> #loadDefault
>>
>> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
>> 11165). Before and after loading O2, running OB tests give "548 run,
>> 547 passes, 1 failures" [*].
>>
>> Cheers,
>> Alexandre
>>
>> [*] the test have been loaded using (ConfigurationOfOmniBrowser
>> project version: '1.1')  load: 'Dev Tests'
>>
>
> OR2PackageBrowser does not open, it lacks a defaultRootNode method
> returning other than nil.
> Is the focus mainly on the O2* browsers, or should OR2* also work?
>
> Cheers,
> Henry
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

David Röthlisberger-2
In reply to this post by Henrik Sperre Johansen

>
> OR2PackageBrowser does not open, it lacks a defaultRootNode method
> returning other than nil.
> Is the focus mainly on the O2* browsers, or should OR2* also work?

How did you spot this problem? From within the O2 browser, OR2PackageBrowser works.
OR2PackageBrowser is not meant to be used independently from the O2 browser.
Something like 'OR2PackageBrowser open' is not supposed to work, as the OR2PackageBrowser
gets the root node from the O2 browser, so there is no reasonable default root node.

Or do you see other problems when using OR2PackageBrowser from within O2?

David

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Henrik Sperre Johansen

On Jan 21, 2010, at 2:30 18PM, David Röthlisberger wrote:

>
>>
>> OR2PackageBrowser does not open, it lacks a defaultRootNode method
>> returning other than nil.
>> Is the focus mainly on the O2* browsers, or should OR2* also work?
>
> How did you spot this problem? From within the O2 browser, OR2PackageBrowser works.
> OR2PackageBrowser is not meant to be used independently from the O2 browser.
> Something like 'OR2PackageBrowser open' is not supposed to work, as the OR2PackageBrowser
> gets the root node from the O2 browser, so there is no reasonable default root node.
>
> Or do you see other problems when using OR2PackageBrowser from within O2?
>
> David
I did OR2PackageBrowser open directly, thinking it was a standalone browser. :/
Not quite sure how/when I use it from within O2, but the refactoring items I've tried in O2PackageBrowser seem to work :)

Quick question: How do you open a browser of methods referring an instance variable from the O2PackageBrowser?

Cheers,
Henry
_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Stan Shepherd
Henrik Sperre Johansen wrote
 but the refactoring items I've tried in O2PackageBrowser seem to work :)
Indeed, nice job. That would remove my reason for sticking to the conventional browser.

Just one (no two) tiny UI issues:

if you right click a package  ->  Refactoring scope ,  -> *package* is greyed out.
If you do the same from one of the classes of the package, it works.

Am I right in thinking that a method name is italic iff it comes from a trait? Should it also be in a dynamic protocol called traits, or similar, and/or an icon? It shows an icon for override of superclass method, for which there is bubble help text, but there is no comparable way to find out where the trait method comes from.
Cheers,
...Stan
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

David Röthlisberger-2
In reply to this post by Adrian Lienhard
Hi Adrian,

> However, there are 6 undeclared classes referenced in the code.

ok, I fixed these undeclared references.

> Also 2 tests have errors and one fails.

Which tests fail? I only got one error in a test which I fixed now.

Cheers,
David


>
> On Jan 19, 2010, at 23:11 , Mariano Martinez Peck wrote:
>
>> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel <
>> [hidden email]> wrote:
>>
>>> Dear All,
>>>
>>> For people who do not know, O2 is the browser David has implemented in
>>> the last few years. It replaces class-categories on the left-hand side
>>> pane with a list of packages. We are still away from having packages
>>> as real first class entities, but this is a good step toward. Note
>>> that some people find O2 a bit slower than other available code
>>> browsers.
>>>
>>> David and I spent some time on fixing O2. O2 has now no dependencies
>>> to OB anymore.
>>> We would like that people try it and submit bug reports on the google
>>> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>>>
>>> Gofer new squeaksource: 'MetacelloRepository'; package:
>>> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:
>>> #loadDefault
>>>
>>>
>> Yes, but be aware that that piece of code load only the core, as you can see
>> here:
>>
>>        group: 'default' with: #('Core' );
>>            group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic'
>> 'O2-Enhancements' );
>>            group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' );
>>
>>
>> As I remember, most of the problems of O2 were sometimes related to the
>> integration to RB. So, if you really want to tests it, please, test it using
>> the RB:
>>
>> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev'
>>
>>
>> As I already said to Alexandre and David in private, I will put O2 again in
>> the next image. So, it would be cool if it is tested "completely".
>>
>>
>>
>>> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
>>> 11165). Before and after loading O2, running OB tests give "548 run,
>>> 547 passes, 1 failures" [*].
>>>
>>>
>> Yes, if someone can help with that failure would be cool, I have already
>> reported it.
>>
>>
>>
>>> Cheers,
>>> Alexandre
>>>
>>> [*] the test have been loaded using (ConfigurationOfOmniBrowser
>>> project version: '1.1')  load: 'Dev Tests'
>>> --
>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>>> Alexandre Bergel  http://www.bergel.eu
>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>>
>>>
>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

Adrian Lienhard
Hi David,

in my PharoCore-1.0-10505 image I had #testSmartGroupSwitch that failed and #testExtensionNodes and #testStyling that errored.

Cheers,
Adrian

On Jan 21, 2010, at 16:09 , David Röthlisberger wrote:

> Hi Adrian,
>
>> However, there are 6 undeclared classes referenced in the code.
>
> ok, I fixed these undeclared references.
>
>> Also 2 tests have errors and one fails.
>
> Which tests fail? I only got one error in a test which I fixed now.
>
> Cheers,
> David
>
>
>>
>> On Jan 19, 2010, at 23:11 , Mariano Martinez Peck wrote:
>>
>>> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel <
>>> [hidden email]> wrote:
>>>
>>>> Dear All,
>>>>
>>>> For people who do not know, O2 is the browser David has implemented in
>>>> the last few years. It replaces class-categories on the left-hand side
>>>> pane with a list of packages. We are still away from having packages
>>>> as real first class entities, but this is a good step toward. Note
>>>> that some people find O2 a bit slower than other available code
>>>> browsers.
>>>>
>>>> David and I spent some time on fixing O2. O2 has now no dependencies
>>>> to OB anymore.
>>>> We would like that people try it and submit bug reports on the google
>>>> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>>>>
>>>> Gofer new squeaksource: 'MetacelloRepository'; package:
>>>> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:
>>>> #loadDefault
>>>>
>>>>
>>> Yes, but be aware that that piece of code load only the core, as you can see
>>> here:
>>>
>>>       group: 'default' with: #('Core' );
>>>           group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic'
>>> 'O2-Enhancements' );
>>>           group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' );
>>>
>>>
>>> As I remember, most of the problems of O2 were sometimes related to the
>>> integration to RB. So, if you really want to tests it, please, test it using
>>> the RB:
>>>
>>> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev'
>>>
>>>
>>> As I already said to Alexandre and David in private, I will put O2 again in
>>> the next image. So, it would be cool if it is tested "completely".
>>>
>>>
>>>
>>>> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
>>>> 11165). Before and after loading O2, running OB tests give "548 run,
>>>> 547 passes, 1 failures" [*].
>>>>
>>>>
>>> Yes, if someone can help with that failure would be cool, I have already
>>> reported it.
>>>
>>>
>>>
>>>> Cheers,
>>>> Alexandre
>>>>
>>>> [*] the test have been loaded using (ConfigurationOfOmniBrowser
>>>> project version: '1.1')  load: 'Dev Tests'
>>>> --
>>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>>>> Alexandre Bergel  http://www.bergel.eu
>>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> Pharo-project mailing list
>>>> [hidden email]
>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] O2: tester wanted!

David Röthlisberger-2

> in my PharoCore-1.0-10505 image I had #testSmartGroupSwitch that failed and #testExtensionNodes and #testStyling that errored.

ah, then probably I fixed the other test methods by fixing the undeclared references.

Thanks,
David

> On Jan 21, 2010, at 16:09 , David Röthlisberger wrote:
>
>> Hi Adrian,
>>
>>> However, there are 6 undeclared classes referenced in the code.
>> ok, I fixed these undeclared references.
>>
>>> Also 2 tests have errors and one fails.
>> Which tests fail? I only got one error in a test which I fixed now.
>>
>> Cheers,
>> David
>>
>>
>>> On Jan 19, 2010, at 23:11 , Mariano Martinez Peck wrote:
>>>
>>>> On Tue, Jan 19, 2010 at 11:03 PM, Alexandre Bergel <
>>>> [hidden email]> wrote:
>>>>
>>>>> Dear All,
>>>>>
>>>>> For people who do not know, O2 is the browser David has implemented in
>>>>> the last few years. It replaces class-categories on the left-hand side
>>>>> pane with a list of packages. We are still away from having packages
>>>>> as real first class entities, but this is a good step toward. Note
>>>>> that some people find O2 a bit slower than other available code
>>>>> browsers.
>>>>>
>>>>> David and I spent some time on fixing O2. O2 has now no dependencies
>>>>> to OB anymore.
>>>>> We would like that people try it and submit bug reports on the google
>>>>> bug tracker. In a fresh 10505, you can load O2 by evaluating:
>>>>>
>>>>> Gofer new squeaksource: 'MetacelloRepository'; package:
>>>>> 'ConfigurationOfO2'; load. (Smalltalk at: #ConfigurationOfO2) perform:
>>>>> #loadDefault
>>>>>
>>>>>
>>>> Yes, but be aware that that piece of code load only the core, as you can see
>>>> here:
>>>>
>>>>       group: 'default' with: #('Core' );
>>>>           group: 'Core' with: #( 'OmniBrowser2' 'O2-Standard' 'O2-Morphic'
>>>> 'O2-Enhancements' );
>>>>           group: 'Dev' with: #( 'Core' 'O2-Refactory' 'OCForO2' );
>>>>
>>>>
>>>> As I remember, most of the problems of O2 were sometimes related to the
>>>> integration to RB. So, if you really want to tests it, please, test it using
>>>> the RB:
>>>>
>>>> ((Smalltalk at: #ConfigurationOfO2) project version: '1.0') load: 'Dev'
>>>>
>>>>
>>>> As I already said to Alexandre and David in private, I will put O2 again in
>>>> the next image. So, it would be cool if it is tested "completely".
>>>>
>>>>
>>>>
>>>>> O2 does not need OB whatsoever. O2 loads in a Core (I tried in a
>>>>> 11165). Before and after loading O2, running OB tests give "548 run,
>>>>> 547 passes, 1 failures" [*].
>>>>>
>>>>>
>>>> Yes, if someone can help with that failure would be cool, I have already
>>>> reported it.
>>>>
>>>>
>>>>
>>>>> Cheers,
>>>>> Alexandre
>>>>>
>>>>> [*] the test have been loaded using (ConfigurationOfOmniBrowser
>>>>> project version: '1.1')  load: 'Dev Tests'
>>>>> --
>>>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>>>>> Alexandre Bergel  http://www.bergel.eu
>>>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Pharo-project mailing list
>>>>> [hidden email]
>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>>>
>>>> _______________________________________________
>>>> Pharo-project mailing list
>>>> [hidden email]
>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project