8000 Fixed the scatter part of fan by paalge · Pull Request #283 · vtsuperdarn/davitpy · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on May 13, 2020. It is now read-only.

Fixed the scatter part of fan #283

Merged
merged 2 commits into from
Jan 17, 2017
Merged

Fixed the scatter part of fan #283

merged 2 commits into from
Jan 17, 2017

Conversation

paalge
Copy link
Contributor
@paalge paalge commented Oct 26, 2016

Plotting was not done on an axis, meaning that the plotting crashed giving the error message:

File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/davitpy-0.5-py2.7-linux-x86_64.egg/davitpy/pydarn/plotting/fan.py", line
345, in plotFan
myFig.scatter(xctr, y[1] * (.98 - w * .025), s=.1 * pts[w],
AttributeError: 'Figure' object has no attribute 'scatter'

Testing can be done with
import datetime
import os
import matplotlib.pyplot as plt
from davitpy import pydarn
pydarn.plotting.fan.plotFan(datetime.datetime(2013,3,16,16,30),['fhe','fhw'],param='power',gsct=False,coords='mag',fill=False)

Plotting was not done on an axis, meaning that the plotting crashed
@paalge paalge closed this Nov 10, 2016
@paalge paalge deleted the scatter branch November 10, 2016 13:12
@ksterne
Copy link
Contributor
ksterne commented Nov 10, 2016

Hello @paalge,

Sorry we weren't able to test this pull request here, did you mean to close the pull request? Without creating a branch on the vtsuperdarn davitpy repository, it can be hard to test this pull request.

@paalge paalge restored the scatter branch November 10, 2016 15:57
@paalge paalge reopened this Nov 10, 2016
@paalge
Copy link
Contributor Author
paalge commented Nov 10, 2016

Sorry for making a mess, somehow I thought this was merged into develop, and removed the branch.
It is now restored

@asreimer
Copy link
Contributor
asreimer commented Jan 17, 2017

Hey @paalge

I've confirmed the bug that you said was there and that your pull request fixes the bug.

Thank you for contributing!

@asreimer asreimer merged commit fbe3a74 into vtsuperdarn:develop Jan 17, 2017
@asreimer asreimer added the bug label Jan 17, 2017
@paalge paalge deleted the scatter branch January 17, 2017 09:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0