SuperDARN / pydarn

Python library for visualizing SuperDARN Data
GNU Lesser General Public License v3.0
31 stars 11 forks source link

EHN: Summary plot defaults - change default Nave line to blue and watermark to False #341

Closed carleyjmartin closed 11 months ago

carleyjmartin commented 1 year ago

Scope

This PR changes the default Nave line in summary plots to be blue. Watermark is now default off.

issue: #324

Approval

Number of approvals: 2

Test

matplotlib version: 3.7.1 Note testers: please indicate what version of matplotlib you are using

Test watermark=True as well.

import matplotlib.pyplot as plt 
import pydarn

file = "/Users/carley/Documents/data/20211102.0000.00.rkn.a.fitacf"
SDarn_read = pydarn.SuperDARNRead(file)
fitacf_data = SDarn_read.read_fitacf()
a = pydarn.RTP.plot_summary(fitacf_data , beam_num=13)
plt.show()

Gives: Screenshot 2023-06-26 at 11 18 41 AM

hiyadutuje commented 1 year ago

The nave line is black and watermark is still appearing when I use the above code. When I include "a = pydarn.RTP.plot_summary(fitacf_data , beam_num=13, watermark = False)", it disappears. I don't know how to change the nave color. I am using matplotlib 3.7.1.

carleyjmartin commented 1 year ago

Thanks for testing Alicreance!

I just want to check, have you uninstalled pydarn from last time you tested and installed this branch ehn/summary-plot-defaults to test the changes?

You can install using the process we emailed about a few weeks ago but substituting in this branch name instead, or you can install using pip3 install git+https://github.com/superdarn/pydarn@ehn/summary-plot-default

Give me an email if you have any questions, I'm still working out in the field so it might take me a few days to reply!

hiyadutuje commented 1 year ago

Dear Carley, No, I didn't.I will, thanks. Alicreance On Thursday, 31 August 2023 at 20:32:31 GMT+2, Carley @.***> wrote:

Thanks for testing Alicreance!

I just want to check, have you uninstalled pydarn from last time you tested and installed this branch ehn/summary-plot-defaults to test the changes?

You can install using the process we emailed about a few weeks ago but substituting in this branch name instead, or you can install using pip3 install @.***/summary-plot-default

Give me an email if you have any questions, I'm still working out in the field so it might take me a few days to reply!

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>