New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid vector symbol size calculation for horizontal lines #4497

Closed
wants to merge 1 commit into
base: branch-6-2
from

Conversation

Projects
None yet
2 participants
@tbonfort
Member

tbonfort commented Oct 17, 2012

Using the UV test in msautotest, we can see that the use of POLAROFFSET seems to be broken for some arrows. I need to investigate in this before 6.2 release. Perhaps this is simply a mapfile issue, but need to be sure. A test also need to be added for POLAROFFSET @tbonfort

@ghost ghost assigned aboudreault Oct 16, 2012

@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort
Member

tbonfort commented Oct 17, 2012

c.f. #3511

@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort

tbonfort Oct 17, 2012

Member

this one is due to the definition of the arrow body that is ill-defined as a vector symbol. For rendering a horizontal line with a vector symbol, this should be used:

points
 0 0
 1 0
end

instead of

points
 0 0
 5 0

The proposed patch fixes the issue, however this will need some more validation. Demilestoning this for now as there's a workaround.

Member

tbonfort commented Oct 17, 2012

this one is due to the definition of the arrow body that is ill-defined as a vector symbol. For rendering a horizontal line with a vector symbol, this should be used:

points
 0 0
 1 0
end

instead of

points
 0 0
 5 0

The proposed patch fixes the issue, however this will need some more validation. Demilestoning this for now as there's a workaround.

@ghost ghost assigned tbonfort Nov 14, 2012

tbonfort added a commit that referenced this pull request Dec 14, 2012

@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort

tbonfort Dec 14, 2012

Member

applied to master, closing

Member

tbonfort commented Dec 14, 2012

applied to master, closing

@tbonfort tbonfort closed this Dec 14, 2012

mkofahl pushed a commit to faegi/mapserver that referenced this pull request Apr 9, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment