{"action":"created","issue":{"url":"https://api.github.com/repos/vtsuperdarn/davitpy/issues/151","labels_url":"https://api.github.com/repos/vtsuperdarn/davitpy/issues/151/labels{/name}","comments_url":"https://api.github.com/repos/vtsuperdarn/davitpy/issues/151/comments","events_url":"https://api.github.com/repos/vtsuperdarn/davitpy/issues/151/events","html_url":"https://github.com/vtsuperdarn/davitpy/pull/151","id":107200178,"number":151,"title":"Field-of-View and HWM14","user":{"login":"aburrell","id":7045886,"avatar_url":"https://avatars.githubusercontent.com/u/7045886?v=3","gravatar_id":"","url":"https://api.github.com/users/aburrell","html_url":"https://github.com/aburrell","followers_url":"https://api.github.com/users/aburrell/followers","following_url":"https://api.github.com/users/aburrell/following{/other_user}","gists_url":"https://api.github.com/users/aburrell/gists{/gist_id}","starred_url":"https://api.github.com/users/aburrell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aburrell/subscriptions","organizations_url":"https://api.github.com/users/aburrell/orgs","repos_url":"https://api.github.com/users/aburrell/repos","events_url":"https://api.github.com/users/aburrell/events{/privacy}","received_events_url":"https://api.github.com/users/aburrell/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":19,"created_at":"2015-09-18T13:14:43Z","updated_at":"2015-10-22T16:24:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/vtsuperdarn/davitpy/pulls/151","html_url":"https://github.com/vtsuperdarn/davitpy/pull/151","diff_url":"https://github.com/vtsuperdarn/davitpy/pull/151.diff","patch_url":"https://github.com/vtsuperdarn/davitpy/pull/151.patch"},"body":"This branch contains new processing routines to determine the origin field-of-view and propagation path for backscatter, as well as an update to the most recent version of the Horizontal Wind Model.\r\n\r\nTests for these changes are included in the davitpy.pydarn.proc.fov.test_update_backscatter module.  This module contains routines to reproduce all of the figures in the paper submitted to Radio Science.  The second to the last figure demonstrates how changes to the overlay plotting routines were needed to allow radar annotation and beam shading when using subplots.  The last figure uses HWM14.\r\nSince it's not out yet, here are the routines to run to reproduce these figures:\r\n\r\nout = davitpy.pydarn.proc.fov.test_update_backscatter.plot_yeoman_plate1()\r\n![han_pyk_heater_eval_elv](https://cloud.githubusercontent.com/assets/7045886/9959887/24577a60-5e0f-11e5-9421-241340927306.png)\r\n\r\nout = davitpy.pydarn.proc.fov.test_update_backscatter.plot_milan_figure9()\r\n![han_groundscatter_eval](https://cloud.githubusercontent.com/assets/7045886/9959913/383df02c-5e0f-11e5-879c-d671bfa0049c.png)\r\n\r\nout = davitpy.pydarn.proc.fov.test_update_backscatter.plot_storm_figures()\r\n\r\n![pyk_vel_test_spectral](https://cloud.githubusercontent.com/assets/7045886/9959922/50586b88-5e0f-11e5-839e-2e12c5869cb9.png)\r\n![pyk_vel_map_spectral](https://cloud.githubusercontent.com/assets/7045886/9959924/527aa534-5e0f-11e5-8c66-43e10c1b42f0.png)\r\n\r\nout = davitpy.pydarn.proc.fov.test_update_backscatter.plot_meteor_figure()\r\n![sas_mscatter_hwm_comp](https://cloud.githubusercontent.com/assets/7045886/9959940/67e61052-5e0f-11e5-9b9d-f6ff648184dd.png)\r\n\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/vtsuperdarn/davitpy/issues/comments/150279553","html_url":"https://github.com/vtsuperdarn/davitpy/pull/151#issuecomment-150279553","issue_url":"https://api.github.com/repos/vtsuperdarn/davitpy/issues/151","id":150279553,"user":{"login":"aburrell","id":7045886,"avatar_url":"https://avatars.githubusercontent.com/u/7045886?v=3","gravatar_id":"","url":"https://api.github.com/users/aburrell","html_url":"https://github.com/aburrell","followers_url":"https://api.github.com/users/aburrell/followers","following_url":"https://api.github.com/users/aburrell/following{/other_user}","gists_url":"https://api.github.com/users/aburrell/gists{/gist_id}","starred_url":"https://api.github.com/users/aburrell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aburrell/subscriptions","organizations_url":"https://api.github.com/users/aburrell/orgs","repos_url":"https://api.github.com/users/aburrell/repos","events_url":"https://api.github.com/users/aburrell/events{/privacy}","received_events_url":"https://api.github.com/users/aburrell/received_events","type":"User","site_admin":false},"created_at":"2015-10-22T16:24:41Z","updated_at":"2015-10-22T16:24:41Z","body":"It’s a very large period of time to run, but I’m surprised you couldn’t load the pickle.  It takes about a minute on my machine.  The pickle cuts out half the computer running time, so I hesitate to give you the whole fitacf.\n\nThe error looks like you haven’t compiled HWM14 yet.  As with any of the models, the least finicky way is to run setup.py again.  It might be fine to go in and use the local make, though, and not go through all the hassle.\n\nTo test hwm14 without running the meteor test, you can just do:\n\nimport davitpy\nimport datetime as dt\n\nstime = dt.datetime(2001, 12, 14)\nalt = 100\nlat = 0\nlon = 0\nap = 4\nihwm = davitpy.models.hwm.hwm_input.format_hwm_input(stime, alt, lat, lon, ap)\ndavitpy.models.hwm.hwm14(*ihwm)\n\n> Out: array([ 23.58708382, -25.50555992], dtype=float32)\n\n\n> On 22 Oct 2015, at 16:45, Muhammad Rafiq <notifications@github.com> wrote:\n> \n> Angeline,\n> \n> I could not be able to test plot_meteor_figure. The pickle file is so large\n> that the command *beams = pickle.load(open(“meteor_ablation_fov.pickle”,\n> “rb”)) *ran on my machine around two hours. After that running *out =\n> davitpy.pydarn.proc.fov.test_update_backscatter.plot_meteor_figure(beams=beams)*\n> gave me this error:\n> \n> \n> *Can not find file\n> /usr/local/lib/python2.7/dist-packages/davitpy-0.2-py2.7-linux-x86_64.egg/davitpy/models/hwm/hwm123114.bin*\n> I have no idea what this *hwm123114.bin *file is. I want to see whether\n> Ashton's testing would also be the same.\n> \n> \n> \n> On Thu, Oct 22, 2015 at 11:31 AM, Angeline Burrell <notifications@github.com\n> > wrote:\n> \n> > Thanks for putting it on your list.\n> >\n> > The miscellaneous changes were all needed to create the plots for the\n> > paper (the ones created in test_update_backscatter).\n> >\n> > 1) The modification was needed for anything else to work. I’m a bit\n> > surprised #141 hasn’t been pushed yet.\n> >\n> > 2) There’s a modification similar to (4), which allows a user to specify\n> > and input axis.\n> >\n> > 3) The important change here (needed to run test_update_backscatter) is:\n> > A) the addition of another parameter (virtual height)\n> > B) changing the way elevation is identified, to allow names that contain\n> > “elv” to all be labeled elevation.\n> >\n> > I think we should change the labels and units to be publication ready. It\n> > wouldn’t be hard to expand the dictionaries to have several options (eg\n> > long-form names, symbols, AGU unit format, some other unit format). The\n> > alternative is for people to not use this dictionary when making their real\n> > plots, which I think would be a waste.\n> >\n> > 5) .gitignore is changed to ignore autosaved python source files (*.py#).\n> > The default just ignores the (*~) files. I think it’s just an oversight\n> > that this ignore wasn’t there already, since most development file types\n> > are already ignored.\n> >\n> > I thought it might be useful to include the rc_parameters file in\n> > gitignore, since those are edited by everyone individually, but I think\n> > that’s overkill.\n> >\n> >\n> > > On 21 Oct 2015, at 20:46, asreimer <notifications@github.com> wrote:\n> > >\n> > > I'll give this code a test in the next couple days.\n> > >\n> > > In the mean time, there are many miscellaneous changes included in this\n> > pull request. They don't seem to fall under the category of FOV and/or\n> > HWM14, so I thought they should be at least highlighted but I have some\n> > questions too. I'll enumerate them:\n> > >\n> > > 1) davitpy/__init__.py is modified (I thought this was to be added to\n> > the rc_parameter pull request #141 <\n> > https://github.com/vtsuperdarn/davitpy/pull/141>)\n> > > 2) davitpy/pydarn/plotting/mapOverlay.py some of this is PEP8, but what\n> > else is happening?\n> > > 3) davitpy/pydarn/radar/radUtils.py label changes for plotting routines.\n> > We should have a discussion on what we want for labelling in DaViTpy. I\n> > remember you brought up suggestions for plotting labels on the plotACF pull\n> > request (#124 (comment) <\n> > https://github.com/vtsuperdarn/davitpy/pull/124#issuecomment-68890436>).\n> > It's probably time we all decided what to do about plot labels (is there\n> > consistent requirements for different journals? If not, how can we make the\n> > labels \"publication ready\"? etc.) This should be it's own pull request.\n> > > 4) davitpy/utils/plotUtils.py - a nice edit to allow a user input axis\n> > object to be used.\n> > > 5) .gitignore is changed. We don't really use this, and it shouldn't be\n> > a problem if people are using git properly, ie) using git add <file>\n> > instead of git commit -a. But since we are on the topic, should we be using\n> > it? What else should go in here?\n> > >\n> > > —\n> > > Reply to this email directly or view it on GitHub <\n> > https://github.com/vtsuperdarn/davitpy/pull/151#issuecomment-150005370>.\n> >\n> > >\n> >\n> > —\n> > Reply to this email directly or view it on GitHub\n> > <https://github.com/vtsuperdarn/davitpy/pull/151#issuecomment-150263652>.\n> >\n> —\n> Reply to this email directly or view it on GitHub <https://github.com/vtsuperdarn/davitpy/pull/151#issuecomment-150267355>.\n> \n\n"}}