Issue 938 in moose-technology: RadialTreeLayout should translate the graph

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

Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology
Status: New
Owner: ----
Labels: Type-Defect Priority-Medium Component-Roassal

New issue 938 by [hidden email]: RadialTreeLayout should translate  
the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

Try this:
view nodes: (Collection withAllSubclasses).
view edgesFrom: #superclass.
view radialTreeLayout.


The center of the graph is placed at 0@0, and this is annoying because you  
do not see half of the graph. The layout should translate the graph to make  
it start at 0@0

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology
Updates:
        Status: Started

Comment #1 on issue 938 by [hidden email]: RadialTreeLayout should  
translate the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

This should not belong to the layout I have the impression. Where to  
position the camera against the nodes the view contains is the real  
question in my opinion. Probably that the other layout are not doing it in  
the right way.

I have added ROFocusView class>>centerView:
The Mondrian builder has been updated accordingly.
For example:
-=-=-=-=-=-=-=-=-=
view nodes: (Collection withAllSubclasses).
view edgesFrom: #superclass.
view radialTreeLayout.
view center.
-=-=-=-=-=-=-=-=-=

produces what is expected.

In Roassal 1.356.
Doru, if you agree with this, then close the issue.

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology

Comment #2 on issue 938 by [hidden email]: RadialTreeLayout should  
translate the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

I think the ability to center the view is nice, but it does not solve the  
layout problem.

The layout positions nodes. So, there is no reason why the layout should  
not start positioning nodes at 0@0. And there is no reason for the default  
camera to not start at 0@0.

Or am I wrong?

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology
Updates:
        Cc: [hidden email]

Comment #3 on issue 938 by [hidden email]: RadialTreeLayout should  
translate the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

Any taker? :)

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology

Comment #4 on issue 938 by [hidden email]: RadialTreeLayout should  
translate the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

Actually, I think the radial tree layout does the right think.  0 @ 0  
should be at the middle of the window. No reason to have it top left corner.
The way layouts are implemented, prevent them from being composed (e.g.,  
there is no easy way to layout a tree with a root node that is not in 0 @  
0).

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology

Comment #5 on issue 938 by [hidden email]: RadialTreeLayout should  
translate the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

I do not get it. I am talking about the user, not the internal  
implementation.

I apply the layout, and two thirds of the graph cannot be seen. How is that  
the right thing?

The only thing that would be required would be to perform a translation of  
the whole graph (or maybe only of the camera) after the layout algorithm  
finished. This would be a linear algorithm depending on the number of nodes.

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology

Comment #6 on issue 938 by [hidden email]: RadialTreeLayout should  
translate the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

Ping? :)

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology

Comment #7 on issue 938 by [hidden email]: RadialTreeLayout should  
translate the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

Mathieu and I are working on this... We will need a few days more...

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology
Updates:
        Status: Fixed

Comment #8 on issue 938 by [hidden email]: RadialTreeLayout should  
translate the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

Sorry to have been that slow on this.

| rawView nodes edges |
rawView := ROView new.

nodes := (ROEllipse gray size: 10) elementsOn: Collection withAllSubclasses.

nodes do: [ :n | n @ RODraggable ].

rawView addAll: nodes.

edges := ROLine buildEdgesFromElements: nodes from: #superclass to:  
#yourself.

rawView addAll: edges.

RORadialTreeLayout new
        "translator: (ROSmoothLayoutTranslator new nbCycles: 10);"
        applyOn: nodes.
rawView openInWindowSized: 1000 @ 600.

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology

Comment #9 on issue 938 by [hidden email]: RadialTreeLayout should  
translate the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

Forgot to say this is now fixed in Roassal 1.458

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 938 in moose-technology: RadialTreeLayout should translate the graph

moose-technology
Updates:
        Labels: Milestone-4.9

Comment #10 on issue 938 by [hidden email]: RadialTreeLayout should  
translate the graph
http://code.google.com/p/moose-technology/issues/detail?id=938

Great!

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev