Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

standardplots anptime plots ckey is incorrect #7

Open
jaythespacehound opened this issue Dec 18, 2018 · 0 comments
Open

standardplots anptime plots ckey is incorrect #7

jaythespacehound opened this issue Dec 18, 2018 · 0 comments

Comments

@jaythespacehound
Copy link

jaythespacehound commented Dec 18, 2018

Currently amp+phase versus time plots use "ckey src" which is great for the one that produces a plot for the whole experiment.
This is a change from previous plots that showed the polarisations in different colours.
The "zoomed in plot" ($start-5m to +55m) should use "ckey p['RR']=2 p['LL']=3."

We now seem to also be plotting multiple subbands. This should probably be changed back to only one in both plots as this makes them very difficult to interpret/follow.

This will likely require a rewrite for the plot functions to support kwargs.
Additionally command line argument should be replaced to use argparse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant