Web Images Videos Maps News Shopping Gmail more »
Recently Visited Groups | Help | Sign in
Google Groups Home
guess-discuss
Home
+ new post
Pages
Files
About this group
Join this group
Bug in Gem layout
Options
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
   4 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Followup To:
Add Cc | Add Followup-to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers you hear
 
Borg  
View profile  
 More options May 31, 2:23 pm
From: Borg <b...@uu3.net>
Date: Sun, 31 May 2009 11:23:42 -0700 (PDT)
Local: Sun, May 31 2009 2:23 pm
Subject: [1.0.3] Bug in Gem layout
Reply to author | Forward | Print | Individual message | Show original | Report this message | Find messages by this author
Hello.

I've found a BUG in Gem Layout.
When all nodes have loop edges following error appears:
java.lang.ArrayIndexOutOfBoundsException: -1 when you try to do Gem
Layout.

How to reproduce:
Load following .gdf file and try to do Gem Layout:
nodedef> name, label, color, strokecolor, labelvisible
n0,"n1","158,176,255","255,255,255",1
n1,"n2","255,255,255","255,255,255",1
n2,"n3","255,255,255","255,255,255",1
edgedef> node1, node2, width, weight, comment VARCHAR
n0,n0,2.0,2.0,""
n1,n1,2.0,10.0,""
n0,n1,2.0,2.0,""
n2,n2,2.0,100.0,""
n0,n2,2.0,2.0,"test"

Please fix fix fix :)
Please compile using 1.0.3 branch


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Eytan Adar  
View profile  
 More options May 31, 4:08 pm
From: Eytan Adar <eytana...@gmail.com>
Date: Sun, 31 May 2009 13:08:30 -0700
Local: Sun, May 31 2009 4:08 pm
Subject: Re: [1.0.3] Bug in Gem layout
Reply to author | Forward | Print | Individual message | Show original | Report this message | Find messages by this author

I'll take a look, but no promises... I'm in the middle of writing my
dissertation.

A workaround is:

foo = removeSelfLoops()
gemLayout()
add(foo)

-E


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Borg  
View profile  
 More options Jun 1, 3:53 pm
From: Borg <b...@uu3.net>
Date: Mon, 1 Jun 2009 12:53:40 -0700 (PDT)
Local: Mon, Jun 1 2009 3:53 pm
Subject: Re: Bug in Gem layout
Reply to author | Forward | Print | Individual message | Show original | Report this message | Find messages by this author
Yeah.
This happens only when ALL nodes have loops.
If even one node doesnt have loop, it works fine.

I belive its something small.. hidden deep in code :) hope it will be
not so hard to find.

Good luck :)
Borg

On May 31, 10:08 pm, Eytan Adar <eytana...@gmail.com> wrote:


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Borg  
View profile  
 More options Jun 12, 10:58 am
From: Borg <b...@uu3.net>
Date: Fri, 12 Jun 2009 07:58:30 -0700 (PDT)
Local: Fri, Jun 12 2009 10:58 am
Subject: Re: Bug in Gem layout
Reply to author | Forward | Print | Individual message | Show original | Report this message | Find messages by this author
Okey. I was able to fix it.
as I said, it was quite simply.
I tested it and it works fine on few of my layouts.
Basicaly fix just ignores loops.
Here it goes:

Index: GEM.java
===================================================================
--- GEM.java    (revision 2)
+++ GEM.java    (working copy)
@@ -225,7 +225,7 @@
        while (nodeSet.hasNext()) {
            uint = (Integer)nodeSet.next();
            ui = uint.intValue();
-           if (gemProp[ui].in != 0) {
+           if (ui != v && gemProp[ui].in != 0) {
                q.addFirst(uint);
                gemProp[ui].in = gemProp[v].in + 1;
            }

On Jun 1, 9:53 pm, Borg <b...@uu3.net> wrote:


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »

Google Groups - Google Home - Terms of Service - Privacy Policy
©2009 Google