Re: gEDA-user: polygon regression in pcb+gl

2011-02-21 Thread Peter C.J. Clifton

On Feb 21 2011, Ineiev wrote:


Pushed to git-head.

Thanks,
Ineiev


Awesome - welcome to the development team. Feel free to ping me if there 
were any outstanding review issues you wanted me to look at with any of 
your existing patches.


I feel very guilty I've not managed to get more of your work looked at and 
pushed up until this point - I think we now have the perfect solution to 
proceed... I look forward to seeing you land the remaining patches ;)




___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-21 Thread Ineiev
On 2/20/11, Stephen Ecob  wrote:
> On Mon, Feb 21, 2011 at 1:51 PM, Kai-Martin Knaak  wrote:
>> Congrats to your new status!
>> The geda project got a new dev!
>> This is really good news :-)
>
> +1 :-)

Thanks to the whole community!


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Steven Michalske
Woo Hoo!


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Stephen Ecob
On Mon, Feb 21, 2011 at 1:51 PM, Kai-Martin Knaak  wrote:
> Ineiev wrote:
>
>> Pushed to git-head.
>
> Great!
> Congrats to your new status!
> The geda project got a new dev!
> This is really good news :-)

+1 :-)


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Kai-Martin Knaak
Ineiev wrote:

> Pushed to git-head.

Great!
Congrats to your new status!
The geda project got a new dev!
This is really good news :-)

---<)kaimartin(>---
-- 
Kai-Martin Knaak
Email: k...@familieknaak.de
Öffentlicher PGP-Schlüssel:
http://pool.sks-keyservers.net:11371/pks/lookup?search=0x6C0B9F53



___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Ineiev

Kai-Martin Knaak wrote:

Ineiev wrote:


I made a wrong assumption (segs variable should be at least 1).

diff --git a/src/polygon.c b/src/polygon.c


I can confirm, that this patch removes the problem both in git-head 
and in Peters gl version. :-)


Pushed to git-head.

Thanks,
Ineiev


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Ineiev

Kai-Martin Knaak wrote:
I can confirm, that this patch removes the problem both in git-head 
and in Peters gl version. :-)


Thank you!

On minor nit: The wasn't attached but part of the body of the email. 
So I had to copy-pasted it to a patch file. This patch file did not 
apply right away. The reason was, that the body gets mangled on its 
course. Some component seems to have replaced tabs with spaces. 
Unfortunately, one of the matching lines contained a tab...


Sorry; I'll bear the issue in mind in the future.


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Kai-Martin Knaak
Ineiev wrote:

> I made a wrong assumption (segs variable should be at least 1).
> 
> diff --git a/src/polygon.c b/src/polygon.c

I can confirm, that this patch removes the problem both in git-head 
and in Peters gl version. :-)

On minor nit: The wasn't attached but part of the body of the email. 
So I had to copy-pasted it to a patch file. This patch file did not 
apply right away. The reason was, that the body gets mangled on its 
course. Some component seems to have replaced tabs with spaces. 
Unfortunately, one of the matching lines contained a tab...

---<)kaimartin(>---
-- 
Kai-Martin Knaak
Email: k...@familieknaak.de
Öffentlicher PGP-Schlüssel:
http://pool.sks-keyservers.net:11371/pks/lookup?search=0x6C0B9F53



___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Ales Hvezda

[snip]
> Peter could, you'd need to spec the port number :-)

Please do not continue to posting messages to the geda mailing lists
with the output from these commands.  Please move discussion about this
to private e-mail.  Thanks.

-Ales



___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Peter Clifton
On Sun, 2011-02-20 at 14:49 +0100, Karl Hammar wrote:

>  Protocol 2
>  RhostsRSAAuthentication no
>  RSAAuthentication no

For my local machine, I have that, but with:

RSAAuthentication yes

 RSAAuthentication
 Specifies whether pure RSA authentication is allowed.  The
 default is “yes”.  This option applies to protocol version 1
 only.


I'll change it to no, but I presume it is unimportant because we also
specify the use of protocol 2.

-- 
Peter Clifton

Electrical Engineering Division,
Engineering Department,
University of Cambridge,
9, JJ Thomson Avenue,
Cambridge
CB3 0FA

Tel: +44 (0)7729 980173 - (No signal in the lab!)
Tel: +44 (0)1223 748328 - (Shared lab phone, ask for me)


signature.asc
Description: This is a digitally signed message part


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Stephan Boettcher
k...@aspodata.se (Karl Hammar) writes:

> Stephan:
>
>> Do "ssh -v git.gpleda.org" to see which version is used.  Most default
>> sshd installations do not permit protocol version 1.
>
> Can't test that:
>
> $ ssh -v git.gpleda.org
> OpenSSH_5.1p1 Debian-5, OpenSSL 0.9.8o 01 Jun 2010
> debug1: Reading configuration data /etc/ssh/ssh_config
> debug1: Applying options for *
> debug1: Connecting to git.gpleda.org [97.107.141.5] port 22.
> debug1: connect to address 97.107.141.5 port 22: Connection refused
> ssh: connect to host git.gpleda.org port 22: Connection refused
> $
>
> Regards,
> /Karl Hammar

Peter could, you'd need to spec the port number :-)

(stephan)blaulicht:~$ ssh -v -p 5022 git.gpleda.org
OpenSSH_5.5p1 Debian-6, OpenSSL 0.9.8o 01 Jun 2010
debug1: Reading configuration data /home/blaulicht/stephan/.ssh/config
debug1: Applying options for *
debug1: Reading configuration data /etc/ssh/ssh_config
debug1: Applying options for *
debug1: Connecting to git.gpleda.org [97.107.141.5] port 5022.
debug1: Connection established.
debug1: identity file /home/blaulicht/stephan/.ssh/id_rsa type 1
debug1: Checking blacklist file /usr/share/ssh/blacklist.RSA-2048
debug1: Checking blacklist file /etc/ssh/blacklist.RSA-2048
debug1: identity file /home/blaulicht/stephan/.ssh/id_rsa-cert type -1
debug1: identity file /home/blaulicht/stephan/.ssh/id_dsa type -1
debug1: identity file /home/blaulicht/stephan/.ssh/id_dsa-cert type -1
debug1: Remote protocol version 2.0, remote software version OpenSSH_5.1p1 
Debian-5
debug1: match: OpenSSH_5.1p1 Debian-5 pat OpenSSH*
debug1: Enabling compatibility mode for protocol 2.0
debug1: Local version string SSH-2.0-OpenSSH_5.5p1 Debian-6
debug1: SSH2_MSG_KEXINIT sent
debug1: SSH2_MSG_KEXINIT received
debug1: kex: server->client aes128-ctr hmac-md5 none
debug1: kex: client->server aes128-ctr hmac-md5 none
debug1: SSH2_MSG_KEX_DH_GEX_REQUEST(1024<1024<8192) sent
debug1: expecting SSH2_MSG_KEX_DH_GEX_GROUP
debug1: SSH2_MSG_KEX_DH_GEX_INIT sent
debug1: expecting SSH2_MSG_KEX_DH_GEX_REPLY
debug1: checking without port identifier
The authenticity of host '[git.gpleda.org]:5022 ([97.107.141.5]:5022)' can't be 
established.
RSA key fingerprint is d7:07:7f:96:51:11:ab:43:57:0d:fe:92:45:72:e8:93.
Are you sure you want to continue connecting (yes/no)? no
Host key verification failed.
(stephan)blaulicht:~$ 


-- 
Stephan


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Karl Hammar
Stephan:
> Peter Clifton  writes:
> > On Sun, 2011-02-20 at 10:36 +0100, Karl Hammar wrote:
...
> >> Don't you know that protocol version 1 i vulnerable for a
> >> man-in-the-middle attack?
> >
> > No, I didn't know that.
> >
> > Does it require a different type of key to be generated and used, or
> > just removing that option to become secure again?

Specify v.2 in your sshd_config, and generally turn off (just in case)
all v.1 protocol stuff as in:

 Protocol 2
 RhostsRSAAuthentication no
 RSAAuthentication no

Use rsa or dsa in your ssh-keygen:

$ man ssh-keygen | grep -A 3 -e '-t type$'
 -t type
 Specifies the type of key to create.  The possible values are
 ``rsa1'' for protocol version 1 and ``rsa'' or ``dsa'' for proto-
 col version 2.

> id_rsa is a version 2 key.

True.

> The RSAAuthentication may be used for version 1 only, but that does not
> mean specifying it makes ssh to use version 1.

True, but there is no reason for it to be there.

> Do "ssh -v git.gpleda.org" to see which version is used.  Most default
> sshd installations do not permit protocol version 1.

Can't test that:

$ ssh -v git.gpleda.org
OpenSSH_5.1p1 Debian-5, OpenSSL 0.9.8o 01 Jun 2010
debug1: Reading configuration data /etc/ssh/ssh_config
debug1: Applying options for *
debug1: Connecting to git.gpleda.org [97.107.141.5] port 22.
debug1: connect to address 97.107.141.5 port 22: Connection refused
ssh: connect to host git.gpleda.org port 22: Connection refused
$

Regards,
/Karl Hammar

---
Aspö Data
Lilla Aspö 148
S-742 94 Östhammar
Sweden
+46 173 140 57




___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Peter Clifton
On Sun, 2011-02-20 at 13:28 +0100, Stephan Boettcher wrote:
> Peter Clifton  writes:

> id_rsa is a version 2 key.
> 
> The RSAAuthentication may be used for version 1 only, but that does not
> mean specifying it makes ssh to use version 1.
> 
> Do "ssh -v git.gpleda.org" to see which version is used.  Most default
> sshd installations do not permit protocol version 1.

Ok, Version 2 is being used. Should I Remove the RSAAuthentication
option anyway?

-- 
Peter Clifton

Electrical Engineering Division,
Engineering Department,
University of Cambridge,
9, JJ Thomson Avenue,
Cambridge
CB3 0FA

Tel: +44 (0)7729 980173 - (No signal in the lab!)
Tel: +44 (0)1223 748328 - (Shared lab phone, ask for me)


signature.asc
Description: This is a digitally signed message part


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Stephan Boettcher
Peter Clifton  writes:

> On Sun, 2011-02-20 at 10:36 +0100, Karl Hammar wrote:
>> Peter Clifton:
>> ...
>> > generate, and setup .ssh/config with these lines:
>> > 
>> > """
>> > Host git.gpleda.org
>> > 
>> > Port 5022
>> > RSAAuthentication yes
>> > IdentityFile ~/.ssh/keys/id_rsa.gpleda.org
>> > """
>> ...
>> 
>> Don't you know that protocol version 1 i vulnerable for a
>> man-in-the-middle attack?
>
> No, I didn't know that.
>
> Does it require a different type of key to be generated and used, or
> just removing that option to become secure again?

id_rsa is a version 2 key.

The RSAAuthentication may be used for version 1 only, but that does not
mean specifying it makes ssh to use version 1.

Do "ssh -v git.gpleda.org" to see which version is used.  Most default
sshd installations do not permit protocol version 1.

-- 
Stephan



___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Peter Clifton
On Sun, 2011-02-20 at 10:36 +0100, Karl Hammar wrote:
> Peter Clifton:
> ...
> > generate, and setup .ssh/config with these lines:
> > 
> > """
> > Host git.gpleda.org
> > 
> > Port 5022
> > RSAAuthentication yes
> > IdentityFile ~/.ssh/keys/id_rsa.gpleda.org
> > """
> ...
> 
> Don't you know that protocol version 1 i vulnerable for a
> man-in-the-middle attack?

No, I didn't know that.

Does it require a different type of key to be generated and used, or
just removing that option to become secure again?

-- 
Peter Clifton

Electrical Engineering Division,
Engineering Department,
University of Cambridge,
9, JJ Thomson Avenue,
Cambridge
CB3 0FA

Tel: +44 (0)7729 980173 - (No signal in the lab!)
Tel: +44 (0)1223 748328 - (Shared lab phone, ask for me)


signature.asc
Description: This is a digitally signed message part


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-20 Thread Karl Hammar
Peter Clifton:
...
> generate, and setup .ssh/config with these lines:
> 
> """
> Host git.gpleda.org
> 
> Port 5022
> RSAAuthentication yes
> IdentityFile ~/.ssh/keys/id_rsa.gpleda.org
> """
...

Don't you know that protocol version 1 i vulnerable for a
man-in-the-middle attack?

$ man ssh_config | grep -A 5 '\bRSAAuthentication$'
 RSAAuthentication
 Specifies whether to try RSA authentication.  The argument to
 this keyword must be ``yes'' or ``no''.  RSA authentication will
 only be attempted if the identity file exists, or an authentica-
 tion agent is running.  The default is ``yes''.  Note that this
 option applies to protocol version 1 only.

http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2001-1473

Regards,
/Karl Hammar

---
Aspö Data
Lilla Aspö 148
S-742 94 Östhammar
Sweden
+46 173 140 57




___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-19 Thread Peter Clifton
On Sat, 2011-02-19 at 19:02 +, Ineiev wrote:
> > Any ideas why?
> 
> I made a wrong assumption (segs variable should be at least 1).

Neat - you got to a fix quicker than I would have!

Lets make this your first commit once you get a developer account setup.
You will need to provide Ales the public half of an SSH key you
generate, and setup .ssh/config with these lines:

"""
Host git.gpleda.org

Port 5022
RSAAuthentication yes
IdentityFile ~/.ssh/keys/id_rsa.gpleda.org
"""

(Adjust as necessary for where you put your key)

You could of course, just give Ales the public key of a key you already
use, but the host and port 5022 bit is important.

The developer access git URL is ssh://g...@git.gpleda.org/pcb.git

I use this in my pcb/git/.git/config:

[remote "origin"]
url = git://git.gpleda.org/pcb.git
fetch = +refs/heads/*:refs/remotes/origin/*

[remote "gpleda"]
url = ssh://g...@git.gpleda.org/pcb.git
push = for_master:master


Best wishes,

-- 
Peter Clifton

Electrical Engineering Division,
Engineering Department,
University of Cambridge,
9, JJ Thomson Avenue,
Cambridge
CB3 0FA

Tel: +44 (0)7729 980173 - (No signal in the lab!)
Tel: +44 (0)1223 748328 - (Shared lab phone, ask for me)


signature.asc
Description: This is a digitally signed message part


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-19 Thread Ineiev

Peter Clifton wrote:

On Wed, 2011-02-16 at 05:27 +0100, Kai-Martin Knaak wrote:

The current head of his pcb+gl branch omits some polygons in my layouts. 
See the attached screen shots. The working copy of pcb+gl was pulled from 
repo.or.cz on December 6th. Unfortunately, I wasn't able to locate the 
the exact version I pulled, or the commit that started the regression.


Urm - I think the bug got into git HEAD too. I think this commit has
caused a regression, but I'm not quite sure why.

commit 2d8dc8a3a3a55158b4e6278dd9f40588e4111c2d
Author: Ineiev 
Date:   Sun Dec 12 00:28:48 2010 -0500

[PATCH] fix bug 2793480 (vias/arcs-to-polygon clearances)

frac_circle(): introduce radius_adjustment factor to make

the polygon outline the arc rather than connet points on the arc

ArcPolyNoIntersect(): compute number of segments so that

polygon diverges from the arc no more than 0.02 of
required thickness; adjust outer "arc" radius like in frac_circle()


Any ideas why?


I made a wrong assumption (segs variable should be at least 1).

diff --git a/src/polygon.c b/src/polygon.c
index c552f3c..0235495 100644
--- a/src/polygon.c
+++ b/src/polygon.c
@@ -498,8 +498,8 @@ ArcPolyNoIntersect (ArcType * a, BDimension thick)
   ry = MAX (a->Height - half, 0);
   segs = 1;
   if(thick > 0)
-segs = a->Delta * M_PI / 360
-   * sqrt(sqrt((double)rx*rx + (double)ry*ry)/delta_th/2/thick);
+segs = MAX (segs, a->Delta * M_PI / 360
+   * sqrt(sqrt((double)rx*rx + (double)ry*ry)/delta_th/2/thick));
   segs = MAX(segs, a->Delta / ARC_ANGLE);

   ang = a->StartAngle;


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-19 Thread Peter Clifton
On Wed, 2011-02-16 at 05:27 +0100, Kai-Martin Knaak wrote:

> The current head of his pcb+gl branch omits some polygons in my layouts. 
> See the attached screen shots. The working copy of pcb+gl was pulled from 
> repo.or.cz on December 6th. Unfortunately, I wasn't able to locate the 
> the exact version I pulled, or the commit that started the regression.

Urm - I think the bug got into git HEAD too. I think this commit has
caused a regression, but I'm not quite sure why.

commit 2d8dc8a3a3a55158b4e6278dd9f40588e4111c2d
Author: Ineiev 
Date:   Sun Dec 12 00:28:48 2010 -0500

[PATCH] fix bug 2793480 (vias/arcs-to-polygon clearances)

frac_circle(): introduce radius_adjustment factor to make
the polygon outline the arc rather than connet points on the arc

ArcPolyNoIntersect(): compute number of segments so that
polygon diverges from the arc no more than 0.02 of
required thickness; adjust outer "arc" radius like in frac_circle()


Any ideas why?

KMK - can you confirm that reverting this patch fixes the problem?
(Reverting polygon.c from my latest branch with a copy from the checkout
you have which works reveals the above commit is the major change... and
other edits don't seem to be a problem).

-- 
Peter Clifton

Electrical Engineering Division,
Engineering Department,
University of Cambridge,
9, JJ Thomson Avenue,
Cambridge
CB3 0FA

Tel: +44 (0)7729 980173 - (No signal in the lab!)
Tel: +44 (0)1223 748328 - (Shared lab phone, ask for me)


signature.asc
Description: This is a digitally signed message part


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-19 Thread Peter Clifton

> True!  I was just wondering if you ran into the same issues I just
> found with rebasing.
> 
> > > It also makes it nearly impossible to maintain your own
> > > branch based off the pcb+gl branch since it will constantly be
> > > diverging.

Use stgit and rebase yourself - that would work.

> I just have been maintaining my own trivial branch
> (2-line patch that still hasn't made it into git HEAD *grumble*
> https://bugs.launchpad.net/pcb/+bug/699498) based off the pcb+gl
> branch, but that made the downside of rebasing evident.

Patch is pushed ;) Hope it makes life easier.

> Anyway, nothing against Peter C's work or his choice of methods to
> manage his branch.  It's all tradeoffs.  I think rebasing makes some
> things quicker but in the long run is messier since you lose track of
> the exact changes you originally made.

Yes, but we don't particularly want to commit those - the alternative is
re-writing the code repeatedly, and I chose to use stgit instead.

> That's all I have to say except to end with thanks to Peter Clifton for
> creating and maintaining the pcb+gl branch, and I hope it goes to
> mainline soon.

Me to - thanks for the positive comments.

-- 
Peter Clifton

Electrical Engineering Division,
Engineering Department,
University of Cambridge,
9, JJ Thomson Avenue,
Cambridge
CB3 0FA

Tel: +44 (0)7729 980173 - (No signal in the lab!)
Tel: +44 (0)1223 748328 - (Shared lab phone, ask for me)


signature.asc
Description: This is a digitally signed message part


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-18 Thread Colin D Bennett
On Fri, 18 Feb 2011 19:12:36 +0100
Kai-Martin Knaak  wrote:

> Colin D Bennett wrote:
> 
> >  The git history is a litte muddled...
> > 
> > When you say it's a little muddled, do you think that if it wasn't
> > rebased constantly that the history would be clearer?
> 
> Probably. But I am not the one who has to deal with the the
> development tree. 

True!  I was just wondering if you ran into the same issues I just
found with rebasing.

> > It also makes it nearly impossible to maintain your own
> > branch based off the pcb+gl branch since it will constantly be
> > diverging.
> 
> On the other hand, it lets the branch track changes of the main
> trunk. It offers the improvements in main when I use Peters version.

Well, rebasing is only one way to get changes from the mainline
branch (git HEAD).  You can also "merge" from the parent (HEAD), and
for any published branches, merging is generally recommended [1].
Merging doesn't rewrite history like rebasing does. However, if it
makes life easier for Peter C to use _rebasing_ rather than _merging_,
that's great.  I just have been maintaining my own trivial branch
(2-line patch that still hasn't made it into git HEAD *grumble*
https://bugs.launchpad.net/pcb/+bug/699498) based off the pcb+gl
branch, but that made the downside of rebasing evident.

Anyway, nothing against Peter C's work or his choice of methods to
manage his branch.  It's all tradeoffs.  I think rebasing makes some
things quicker but in the long run is messier since you lose track of
the exact changes you originally made.

That's all I have to say except to end with thanks to Peter Clifton for
creating and maintaining the pcb+gl branch, and I hope it goes to
mainline soon.

[1] "The perils of rebasing"
.

Regards,
Colin


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-18 Thread Kai-Martin Knaak
Colin D Bennett wrote:

>  The git history is a litte muddled...
> 
> When you say it's a little muddled, do you think that if it wasn't
> rebased constantly that the history would be clearer?

Probably. But I am not the one who has to deal with the the development
tree. 


> It also makes it nearly impossible to maintain your own
> branch based off the pcb+gl branch since it will constantly be
> diverging.

On the other hand, it lets the branch track changes of the main trunk. 
It offers the improvements in main when I use Peters version.


> Tip:  Enabling antialiasing for pcb+gl.
> Peter C. helped me get this working and it's really nice.
> If you have a relatively powerful nVidia graphics card on Linux, go
> to the nvidia control panel

Tried it. The result was beautiful, but painfully slow. I guess, my GPU is 
relatively powerless. ;-)
On my faster desktop I run ATI cards for political reason...

---<)kaimartin(>---
-- 
Kai-Martin Knaak  tel: +49-511-762-2895
Universität Hannover, Inst. für Quantenoptik  fax: +49-511-762-2211 
Welfengarten 1, 30167 Hannover   http://www.iqo.uni-hannover.de
GPG key:http://pgp.mit.edu:11371/pks/lookup?search=Knaak+kmk&op=get



___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user


Re: gEDA-user: polygon regression in pcb+gl

2011-02-17 Thread Colin D Bennett
On Wed, 16 Feb 2011 05:27:29 +0100
Kai-Martin Knaak  wrote:

> This may be mainly for Peter Clifton:
> The current head of his pcb+gl branch omits some polygons in my
> layouts. See the attached screen shots. The working copy of pcb+gl
> was pulled from repo.or.cz on December 6th. Unfortunately, I wasn't
> able to locate the the exact version I pulled, or the commit that
> started the regression. The git history is a litte muddled...

When you say it's a little muddled, do you think that if it wasn't
rebased constantly that the history would be clearer?  I know there are
good arguments both for and against rebasing, but I have to say rebasing
does make it harder to keep track of history and especially to track
down breakage.  It also makes it nearly impossible to maintain your own
branch based off the pcb+gl branch since it will constantly be
diverging. I'm no git expert (I'm a Bazaar guy) but this is my
understanding and experience.

> Hopefully, this is easy to fix. Let me know, if I can help to locate
> the bug. Other than that, the openGL version is awesome. I get 30 FPS
> with full polygons for a layout that had a mere 10 FPS with polygons
> switched off two years ago (on the same hardware). 

Agreed!  I am really loving the OpenGL pcb.  Thanks, Peter Clifton.
The layer translucency support is worth it alone (a very functional
feature), but there is more as well.

Tip:  Enabling antialiasing for pcb+gl.
Peter C. helped me get this working and it's really nice.
If you have a relatively powerful nVidia graphics card on Linux, go
to the nvidia control panel (Ubuntu: System | Administration |
NVIDIA X Server Settings) and choose "Antialiasing Settings" under
your X screen on the left panel, then choose "Override Application
Settings" and crank up the Antialiasing Settings slider to the max, or
whatever level you prefer.  Pcb is really beautiful with
antialiasing+translucency!

Regards,
Colin


___
geda-user mailing list
geda-user@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user