Closed ksterne closed 6 years ago
Hi Kevin,
Here is the plot I get from your testing code:
This is what I get from the develop branch:
You can see that the legend in the left first panel and the range gate info in the title are gone. The x axis of lag number is also different as suggested by Kevin's comments.
May I suggest two changes? One is to have an additional zero pad for the range gate number on the files, since some radars have a maximum gate of 225. The other is to decrease the font size on the legend (probably specify "medium" or "small") since it looks like it's larger than the axes labels.
Hey @aburrell, thanks for the feedback here. Those are easy things to change up and I've done them now. I've also forced the acf plot's legend to 1 one and 2 columns as I think that will look better on more plots by keeping it away from the data points. What do you think?
This is better. Here's what I get for running the code now:
The last cosmetic changes I think would help are:
1) Make the title two lines, with a "\n" either after the radar name or after the date 2) Remove the comma after the month (AGU date standard is "17 Sep 2017") 3) Change from "m/s" to "m s$^{-1}$", again for plot publishing standards. 4) Top left y-axis is almost off the plot, decreasing the white space between the right and left columns would help.
OK @aburrell, I can take it that there's some basic things to fix up here, but starting to creep away from the original intent. Things look good after the last couple of commits I pushed? Only way to solve 4) was to move all of the plots to the right a little which makes it a little off balance, but still looks good to me.
This looks good to me, I think the output is usable now.
So....is this ready to be merged? @aburrell?
I say yes. Shall I do the honours?
On 16 Feb 2018, at 08:43, Kevin Sterne notifications@github.com wrote:
So....is this ready to be merged? @aburrell https://github.com/aburrell?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/vtsuperdarn/davitpy/pull/334#issuecomment-366253928, or mute the thread https://github.com/notifications/unsubscribe-auth/AGuC_i9f-yNbgQ7xRbeolWskfsrCI99dks5tVZQegaJpZM4RtZpO.
@ksterne , I'll let you decide whether or not to do the branch clean up.
hello @ksterne . I came across your project while I was hunting the gsoc page and got really interested in it. I am pretty comfortable with python and was wondering if I could be a contributor. I read the guidelines, but was a little apprehensive about mailing you directly, ( for the project "Develop open source software in python to download, analyze and visualize space science datasets").
I would be really thankful if you could give a general idea, on a few things: i) when would be the right time to mail you regarding the project ii) what are the expectations from an applicant in general . Suggestions on anything you find is worth mentioning really. I would be grateful to you if you could just give a general advice on how to get started with the project. Thank you
E-mailing me directly would be much more appropriate than commenting on a closed pull request which can be sent to all as an update. Please look at our ideas list tomorrow as our application process will be changing.
I know things are fairly quiet, but here's a fairly minor and simple update I came across when using the acfPlot code. Basically just making the resulting plot look a little prettier and usable. Things that I changed (in case you're too lazy to read the commit messages):
Sample test code:
You should get something like:
Run the same code on the develop and see the differences between the two. Not great, but this one presents the info/data a little better.