-
-
Notifications
You must be signed in to change notification settings - Fork 421
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
Enabling spectrum plotting in the frequency domain #1695
Conversation
Before a pull request is accepted, it must meet the following criteria:
|
Codecov Report
@@ Coverage Diff @@
## master #1695 +/- ##
==========================================
- Coverage 61.29% 61.26% -0.03%
==========================================
Files 63 63
Lines 5792 5798 +6
==========================================
+ Hits 3550 3552 +2
- Misses 2242 2246 +4
Continue to review full report at Codecov.
|
* allowing frequency plotting * docstring beginning * docstring * Get rid of accidental change * fixing latex in axis labels * trying to get axis labels formatted * fixing axis label * Fixing small mistake
Resubmitting #1666
Description
This PR
"frequency"
mode inTARDISSpectrum.plot()
.TARDISSpectrum.plot()
.TARDISSpectrum.plot()
.Motivation and context
Currently, TARDIS cannot automatically plot spectra in the frequency domain.
How has this been tested?
Examples
The following images show the implementation of my changes:
Type of change
Checklist