works with lon range - Githubissues

space-physics / iri2016

International Reference Ionosphere 2016 from Python and Matlab

http://irimodel.org/

MIT License

67 stars 34 forks source link

works with lon range #25

Open kvenkman opened 3 years ago

kvenkman commented 3 years ago

The method now iterates over a given range of longitudes.

A few other additions - glat/glon are checked for sequences/numpy arrays. If they are, the np.arange method used along with unpacking operator now creates a np.float array. This fixes the assertion error raised at line 24 in IRI of base.py if the input sequence comprises only of integers - e.g. (30, 40, 2).

If glat/glon are not Sequences/numpy arrays, they are expanded to be 1D numpy arrays to be iterable.

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 86ce42b924351829ef8744767bfd3d68a49bda51 into f04ded9ead6f4a24bac523041a147d8e16c54049 - view on LGTM.com

new alerts:

1 for Wrong name for an argument in a call

scivision commented 3 years ago

thank you -- I will look at this soon!

kvenkman commented 3 years ago

The call to geoprofile in latitude.py needs to be updated - should I fix this and re-submit a PR?

kvenkman commented 3 years ago

Following up on this PR - is there anything I can do to have this feature included?

scivision commented 3 years ago

Sorry, I got very busy I am catching up on PRs !