changed default colormap and default parameter to velocity

[mts299]

As requested in #24 the new default parameter is v Velocity and the new default colormap is the PydDARN Velocity colormap. To keep with the majority of what users plot and with the velocity colormap convention.

import pydarn
import matplotlib.pyplot as plt 

fitacf_file = "20180101.C0.cly.fitacf"
darn_read = pydarn.SDarnRead(fitacf_file)
fitacf_data = darn_read.read_fitacf()
pydarn.RTP.plot_range_time(fitacf_data, zmin=-600, zmax=600)
plt.show()

Granted this has one caveat with the other parameters:

However, they can change colormaps flexibly:

import pydarn
import matplotlib.pyplot as plt 

fitacf_file = "20180101.C0.cly.fitacf"
darn_read = pydarn.SDarnRead(fitacf_file)
fitacf_data = darn_read.read_fitacf()
pydarn.RTP.plot_range_time(fitacf_data, parameter='p_l', cmap='plasma')
plt.show()

@billetd if you don't mind updating plot_range_time tutorials again to update with the new changes and mention that if users wish to use another colormap for parameters like p_l Signal to Noise with some suggestions.

[asreimer]

Just to clarify, I think the issue I raised was invalid. If it is expected that most people will use plot_range_time to plot velocity first, then this default change makes a lot of sense! I can test and merge when I get to a computer.

…

On Tue, Dec 17, 2019, 08:37 Marina Schmidt ***@***.***> wrote: As requested in #24 <#24> the new default parameter is v Velocity and the new default colormap is the PydDARN Velocity colormap. To keep with the majority of what users plot and with the velocity colormap convention. import pydarn import matplotlib.pyplot as plt fitacf_file = "20180101.C0.cly.fitacf" darn_read = pydarn.SDarnRead(fitacf_file) fitacf_data = darn_read.read_fitacf() pydarn.RTP.plot_range_time(fitacf_data, zmin=-600, zmax=600) plt.show() [image: RTI_velocity] <https://user-images.githubusercontent.com/6520530/71014924-59bf4500-20b8-11ea-8494-d6bb8da1806a.png> Granted this has one caveat with the other parameters: [image: RTI_power] <https://user-images.githubusercontent.com/6520530/71015077-a1de6780-20b8-11ea-9f64-fdb54d287861.png> However, they can change colormaps flexibly: import pydarn import matplotlib.pyplot as plt fitacf_file = "20180101.C0.cly.fitacf" darn_read = pydarn.SDarnRead(fitacf_file) fitacf_data = darn_read.read_fitacf() pydarn.RTP.plot_range_time(fitacf_data, parameter='p_l', cmap='plasma') plt.show() [image: RTI_plasma_SNR] <https://user-images.githubusercontent.com/6520530/71015379-1ca78280-20b9-11ea-941a-8987a41ca8f0.png> @billetd <https://github.com/billetd> if you don't mind updating plot_range_time tutorials again to update with the new changes and mention that if users wish to use another colormap for parameters like p_l Signal to Noise with some suggestions. ------------------------------ You can view, comment on, or merge this pull request online at: #25 Commit Summary - changed default colormap and default parameter to velocity File Changes - *M* pydarn/plotting/rtp.py <https://github.com/SuperDARN/pydarn/pull/25/files#diff-0> (8) Patch Links: - https://github.com/SuperDARN/pydarn/pull/25.patch - https://github.com/SuperDARN/pydarn/pull/25.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#25?email_source=notifications&email_token=ABDEHEYEKA7C73DEXVBTKGTQZD535A5CNFSM4J36IGP2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IBDIHOA>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABDEHE2N7VDSY7A3SFH5SRTQZD535ANCNFSM4J36IGPQ> .

[billetd]

PR #31 on documentation has been updated to include the new colourmap and defaults here, but can't be merged till this is ;).

I've been using this branch for a while now and haven't ran into any issues. I like the default colourmap alot and am happy for default RTP to be velocity.