Giter Club home page Giter Club logo

Comments (10)

jswhit avatar jswhit commented on May 22, 2024

For reasons not yet clear to me, you have to reverse the sign of the vectors that come back from transform_vector for SH polar plots, i.e.

ups = -ups; vps = -vps

in your test script right after the call to transform_vector. Must have something to do with how the projection coordinates are defined in proj4.

from basemap.

joakimkjellsson avatar joakimkjellsson commented on May 22, 2024

Thanks. That solved it. You're probably right in that its not a basemap issue per se but rather a proj4 thing...

from basemap.

efiring avatar efiring commented on May 22, 2024

Still, it seems to me that transform_vector should take care of this transparently, so I am reopening this.

from basemap.

joanpau avatar joanpau commented on May 22, 2024

I think that the reason for this behavior is that when using polar projections at the southern hemisphere the ranges of the axes are negative:

In [30]: m = Basemap(projection='spstere', boundinglat=0, lon_0=0)

In [31]: plt.gca().get_xlim()
Out[31]: (0.0, -25483987.999999996)

In [32]: plt.gca().get_ylim()
Out[32]: (0.0, -25483987.999999996)

combined with the default value of the option angles = 'uv' of Basemap's quiver (inherited from Matplotlib's quiver):

angles: [ ‘uv’ | ‘xy’ | array ]
    With the default ‘uv’, the arrow axis aspect ratio is 1, so that if U*==*V
    the orientation of the arrow on the plot is 45 degrees CCW from the horizontal axis
    (positive to the right). With ‘xy’, the arrow points from (x,y) to (x+u, y+v).
    Use this for plotting a gradient field, for example. Alternatively, arbitrary angles
    may be specified as an array of values in degrees, CCW from the horizontal axis.
    Note: inverting a data axis will correspondingly invert the arrows only with angles='xy'.

When using 'angles=xy', vectors are plotted properly.
This is consistent with the computations in rotate_vector.
However, there is something that I can not understand in that function,
namely code in lines 3137-3142:

        # Define a displacement (dlon, dlat) that moves all
        # positions (lons, lats) a small distance in the
        # direction of the original vector.
        dc = 1E-5 * np.exp(theta*1j)
        dlat = dc.imag * np.cos(np.radians(lats))
        dlon = dc.real

What is the foundation of that computation? Should not this be:

        # Define a displacement (dlon, dlat) that moves all
        # positions (lons, lats) a small distance in the
        # direction of the original vector.
        dc = 1E-5 * np.exp(theta*1j)
        dlat = dc.imag
        dlon = dc.real / np.cos(np.radians(lats))

???

from basemap.

guziy avatar guziy commented on May 22, 2024

Hi:

Yes you are right it should be divided by cos in dlon, but that might cause a division by 0 problem, so I suppose he multiplied the vector (i.e. both components) by cos. And it is OK to multiply since cos(lat) is not negative and preserves direction.

Cheers

from basemap.

joanpau avatar joanpau commented on May 22, 2024

Thanks @guzly. I came to that conclusion too. I just wondered
if there were a different geometric reasoning running here I was not aware of.
Also, from this it can be deduced that the input components u and v
should be in metric units, and not in angular units.
I mean that for the case of velocities they should be in some multiple of m/s
and not deg/s. I was not able to deduce this from the documentation:

The input vector field is defined in spherical coordinates (it
has eastward and northward components) while the output
vector field is rotated to map projection coordinates (relative
to x and y). The magnitude of the vector is preserved.

The wording does not specify it, and other mapping systems use angular units
instead of metric units for vector component inputs
(e.g. quiverm from MATLAB Mapping Toolbox)
I think that this should be more clearly stated in the docstring.
Do you think this deserves a new issue and/or pull request?

from basemap.

guziy avatar guziy commented on May 22, 2024

I think this function would work for any units. Spherical coordinates, as I
understand, here means that velocity components are along meridians and
parallels... But let's see what others think, maybe the documentation would
benefit from additional clarification...

Cheers

2015-09-14 7:28 GMT-04:00 Joan Pau Beltran [email protected]:

Thanks @guzly. I came to that conclusion too. I just wondered
if there were a different geometric reasoning running here I was not aware
of.
Also, from this it can be deduced that the input components u and v
should be in metric units, and not in angular units.
I mean that for the case of velocities they should be in some multiple of
m/s
and not deg/s. I was not able to deduce this from the documentation:

The input vector field is defined in spherical coordinates (it
has eastward and northward components) while the output
vector field is rotated to map projection coordinates (relative
to x and y). The magnitude of the vector is preserved.

The wording does not specify it, and other mapping systems use angular
units
instead of metric units for vector component inputs
(e.g. quiverm from MATLAB Mapping Toolbox
http://www.mathworks.com/help/map/ref/quiverm.html)
I think that this should be more clearly stated in the docstring.
Do you think this deserves a new issue and/or pull request?


Reply to this email directly or view it on GitHub
#157 (comment).

Sasha

from basemap.

joanpau avatar joanpau commented on May 22, 2024

Not actually. If the velocity components are eastward and northward but already expressed in deg/s
the factor np.cos(np.radians(lats)) in the above code should not be there, leaving instead:

        dc = 1E-5 * np.exp(theta*1j)
        dlat = dc.imag
        dlon = dc.real

Since only one component is affected, this is not a scaling: the direction of the
approximated differential increment is different.

I am not asking to change the current behavior to this one.
I am just saying that the expected unit convention of the input vector components (metric vs angular)
could be more explicitly stated in the docstring.

from basemap.

guziy avatar guziy commented on May 22, 2024

Maybe it is a bit tricky to understand... I'll try to explain later or
maybe someone who knows better could help.... I have some deadlines for
tomorrow...

Cheers

2015-09-14 9:00 GMT-04:00 Joan Pau Beltran [email protected]:

Not actually. If the velocity components are eastward and northward but
already expressed in deg/s:
the factor np.cos(np.radians(lats)) in the above code should not be
there, leaving instead:

    dc = 1E-5 * np.exp(theta*1j)
    dlat = dc.imag
    dlon = dc.real

Since only one component is affected, this is not a scaling: the direction
of the
approximated differential increment is different.


Reply to this email directly or view it on GitHub
#157 (comment).

Sasha

from basemap.

guziy avatar guziy commented on May 22, 2024

Actually ...

You are right, sorry)) I hope nobody will try to use it this way....
Because in this case units for the components are different though they
have the same name and in order to calculate the module one of them should
be divided or multiplied by cos ....

Confusing ...))

2015-09-14 9:26 GMT-04:00 Oleksandr Huziy [email protected]:

Maybe it is a bit tricky to understand... I'll try to explain later or
maybe someone who knows better could help.... I have some deadlines for
tomorrow...

Cheers

2015-09-14 9:00 GMT-04:00 Joan Pau Beltran [email protected]:

Not actually. If the velocity components are eastward and northward but
already expressed in deg/s:
the factor np.cos(np.radians(lats)) in the above code should not be
there, leaving instead:

    dc = 1E-5 * np.exp(theta*1j)
    dlat = dc.imag
    dlon = dc.real

Since only one component is affected, this is not a scaling: the
direction of the
approximated differential increment is different.


Reply to this email directly or view it on GitHub
http://t.sidekickopen18.com/e1t/c/5/f18dQhb0S7lC8dDMPbW2n0x6l2B9nMJN7t5XZsf6M1jN2B8hqKdnGrCVd_V9H56dHzydWxrcl02?t=https%3A%2F%2Fgithub.com%2Fmatplotlib%2Fbasemap%2Fissues%2F157%23issuecomment-140063177&si=5057221580292096&pi=9bb774f1-28b6-4766-d4a4-2dcf8506bc70
.

Sasha

Sasha

from basemap.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.