Opened 13 years ago

Closed 13 years ago

#875 closed defect (fixed)

Regress errors: ReprojectPts, ReprojectPtsGeog, tickets

Reported by: aperi2007 Owned by: pramsey
Priority: medium Milestone: PostGIS 2.0.0
Component: postgis Version: master
Keywords: Cc:

Description

Hi, compiling the r6943 on a debian lenny 32bit with postgres 9.0.2 I see these errors:

loader/ReprojectPts….. failed (wkt test: diff expected obtained: /tmp/pgis_reg_8047/test_20_diff)

loader/ReprojectPtsGeog….. failed (wkt test: diff expected obtained: /tmp/pgis_reg_8047/test_21_diff)

tickets. failed (diff expected obtained: /tmp/pgis_reg_8047/test_58_diff)

Attach the logs for pgis_reg_8047.

Attachments (2)

pgis_reg_8047.zip (8.8 KB ) - added by aperi2007 13 years ago.
logs of errors
pgis_reg_21420.zip (5.5 KB ) - added by aperi2007 13 years ago.
log for ticket error

Download all attachments as: .zip

Change History (15)

by aperi2007, 13 years ago

Attachment: pgis_reg_8047.zip added

logs of errors

comment:1 by strk, 13 years ago

Hudson is reporting the same problems (dunno why _to_me_ actually, but that's another story)

comment:2 by jadams, 13 years ago

Can you please attach the contents of /tmp/pgis_reg_8047/test_20_diff and /tmp/pgis_reg_8047/test_21_diff?

Also is there a way I can see the hudson results? I added these tests (so it's probably my fault they're failing) but I didn't realize hudson was complaining.

comment:3 by aperi2007, 13 years ago

it is add to the ticket.

comment:5 by robe, 13 years ago

Those just look like rounding errors. Don't we have some logic in regress to truncate end decimals?

BTW: tickets is failing for me too on MingW - but I suspect its somethin else that changed perhaps something Paul changed. I was blaming Olivier but I don't think he committed anything during the sprint.

comment:6 by jadams, 13 years ago

I agree they look like rounding errors. I had increased the precision of the output, which improved but clearly didn't fix the problem.

There is not anything in universal regress to truncate decimals though it is a good idea.

comment:7 by robe, 13 years ago

ST_SnapToGrid or did you try that already. I think that's how we've dealt with these in the past.

comment:8 by strk, 13 years ago

round(x * 1eY) / 1eY

with Y being number of decimal digits

comment:9 by robe, 13 years ago

I put in an ST_SnapToGrid call and it still works for me. strk and whoever was having problems can you test it and see if it passes for you

comment:10 by nicklas, 13 years ago

Regina, I think it was mostly Hudson complaining. Shortly after your commit Hudson sent a message that it was satisfied.

That is quite strange, isn't it? Why did the ticket issue disappear? And then come back when Pierre removed some issues in raster.

Strange!

/Nicklas

comment:11 by robe, 13 years ago

there are 2 jobs one is the build and the other is the regress (took me a while to figure that out since I noticed the strange thing). The regress is the one that has been failing usually. The build is usually successful — that's why the site docs and tar file on site is up to date.

comment:12 by aperi2007, 13 years ago

Hi, I tested again using geos 3.3.0 trunk r3275 and postgis trunk r6978 on a debian lenny 32bit with postgres 9.0.1.

Some error was disappeared. but still having the ticket error.

tickets. failed (diff expected obtained: /tmp/pgis_reg_21420/test_59_diff)

I put the logs in attach to this ticket.

by aperi2007, 13 years ago

Attachment: pgis_reg_21420.zip added

log for ticket error

comment:13 by robe, 13 years ago

Resolution: fixed
Status: newclosed

well Hudson seems to be happy with my changes so I'll take that as a green light to close this one out. That just leaves — #869 to make the regress completely pass.

Note: See TracTickets for help on using tickets.