Opened 6 years ago
Closed 6 years ago
#3688 closed defect (fixed)
ST_AsLatLonText does not round numbers as expected
|Reported by:||Mike Taves||Owned by:||strk|
SELECT ST_AsLatLonText('POINT (76.6 -76.6)');
While this is technically correct, 60 seconds is not expected. An expected result would be:
Change History (13)
comment:1 by , 6 years ago
comment:2 by , 6 years ago
|Status:||new → assigned|
comment:3 by , 6 years ago
comment:4 by , 6 years ago
|Status:||assigned → closed|
comment:5 by , 6 years ago
|Status:||closed → reopened|
comment:6 by , 6 years ago
This seems to be triggering an eror on winnie. I'm rerunning now but at least for trunk been getting this on the 9.3 and 9.6 32-bit runs:
Suite: printing Test: test_lwprint_default_format ... Assert failed: 76Â°35'60.000"S 76Â°35'60.000"E (actual) 76Â°36'0.000"S 76Â°36'0.000"E (expected) FAILED 1. cu_print.c:33 - CU_ASSERT_STRING_EQUAL(actual,expected)
I'm checking to see if others have the same issue. Given that I didn't run into this issue on my local (and I usually only bother testing 64-bit), I think this might be a 32-bit only issue.
comment:7 by , 6 years ago
|Priority:||medium → blocker|
comment:8 by , 6 years ago
|Priority:||blocker → medium|
just occurred to me the 32-bit answer just returns the old answers so not critical.
comment:9 by , 6 years ago
Any chance we can have this resolved before next Wednesday? I'd like to release 2.3.2 sometime next week and can't be having 32-bits failing.
I guess I could just disable this test on 2.3 though I'd rather not.
comment:10 by , 6 years ago
Regina I've read there's a patch waiting for you (on github?). BTW, I think thre's a jenkins plugin for GitHub, in case you want to try at having Jenkins build the pull requests placed there.
comment:11 by , 6 years ago
Yah I saw the patch, just haven't gotten around to testing it yet. Should later today. I know about the plugin, just not energized enough to fiddle with it.
comment:12 by , 6 years ago
comment:13 by , 6 years ago
|Status:||reopened → closed|