-
Notifications
You must be signed in to change notification settings - Fork 59
OverlayFan polygon doesn't always close #258
Comments
Hey @aburrell, do you have some example code so I can try to reproduce this problem? Does the same thing happen if you only plot 1 fov, or only if you plot 2? |
It’s happened every time I did a groundscatter fov at HAN regardless of the coordinates, number of FOVs, or direction of FOV. I’ll get some example code up at some point.
|
Example code: import datetime as dt fov_test = davitpy.pydarn.radar.radFov.fov(site=hard, ngates=75, altitude=350, coords="geo", model="IS", fov_dir="back") |
The example code fails due the Basemap not having support for datetime objects needed for coordinate conversion:
Modifying the example code to:
I am able to reproduce the above plot. Any idea why this is happening? |
Ok, thing 1) In the original example I had added: m.dateTime=stime Which is why I didn't get an error. But I forgot I added that. |
Ah ok that makes sense. Any idea why the FOV isn't closing on the GS plot? |
Ok, I figured it out. The fov object doesn't save the model it used to create the object, so you have to keep specifying it. |
We should probably add a model to the fov object and have routines check for it. If it doesn't exist, it can default to IS but spit out a warning. |
In [79]: print fov_test ^ |
Awesome! And I agree with that fix. Luckily it's a simple fix too :) |
Yes... Unfortunately I'm heading offline since it's 21:30 and I'm still in the office sans dinner :'( |
Yes! Go home and eat. This can wait. |
As promised! Pull request #269 |
Excellent! I have some things to do this afternoon, but I'll get it tested and merged tonight. |
Issue resolved. |
When you specify the groundscatter option, you the polygon defining the field of view doesn't close:
The text was updated successfully, but these errors were encountered: