Bug #746

The state boundary of Ohio

Added by Chenjian almost 2 years ago. Updated almost 2 years ago.

Status:ClosedStart date:2015-08-08
Priority:NormalDue date:
Assignee:Paul% Done:

100%

Category:-
Target version:Candidate for next bugfix release
Affected version:5.2-svn Platform:

Description

Hi,

The "full" state bound of Ohio in gshhg2.3.4 seems not right at one point. The output is attached.
Script line:

gmt pscoast -JM6i -Ba4/a1WSne -P -R-88/-76/37/44 -Df -EUS.OH,+p0.3,red -A20/1/1 -W0.5p -Glightgreen -Slightblue > pscoast_oh.ps

In addition, when building gmt5.2 r14727, an error popped up:

Building C object src/CMakeFiles/gmtlib.dir/gmt_vector.c.o
/cygdrive/e/gmt52/src/gmt_vector.c: In function ‘GMT_gaussjordan’:
/cygdrive/e/gmt52/src/gmt_vector.c:426:6: error: for statement expected before ‘k’
      k = j; c = fabs(a[k*n+j]);
      ^
src/CMakeFiles/gmtlib.dir/build.make:591: recipe for target 'src/CMakeFiles/gmtlib.dir/gmt_vector.c.o' failed
make[2]: *** [src/CMakeFiles/gmtlib.dir/gmt_vector.c.o] Error 1
CMakeFiles/Makefile2:325: recipe for target 'src/CMakeFiles/gmtlib.dir/all' failed
make[1]: *** [src/CMakeFiles/gmtlib.dir/all] Error 2
Makefile:137: recipe for target 'all' failed
make: *** [all] Error 2

Thanks!

pscoast_oh.png (28.4 KB) Chenjian, 2015-08-08 13:14

Associated revisions

Revision 14737
Added by Remko almost 2 years ago

Move parallelization pragma in GMT_gaussjordan.
This fixes issue #746

History

#1 Updated by Joaquim almost 2 years ago

I think I just fixed the build error in r14729

#2 Updated by Paul almost 2 years ago

  • Status changed from New to In Progress

I will look at the DCW Ohio polygon when I get back from my trip.

#3 Updated by Chenjian almost 2 years ago

Thank you! Right, it should be DCW-GMT 1.1.1, not GSHHG 2.3.4

#4 Updated by Remko almost 2 years ago

r14737 should be fixing the build error better than r14729 did. It would not impact the problem with the OH state boundary.

#5 Updated by Paul almost 2 years ago

  • Status changed from In Progress to Resolved
  • Assignee set to Paul
  • Target version set to Candidate for next bugfix release
  • % Done changed from 0 to 100

Should be fixed in new DCW-GMT version 1.1.2, released today (Sept. 1, 2015). Thanks for pointing it out.

#6 Updated by Paul almost 2 years ago

  • Status changed from Resolved to Closed

Closed as fixed.

Also available in: Atom PDF