-
-
Notifications
You must be signed in to change notification settings - Fork 127
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
new README.rst #116
new README.rst #116
Conversation
There are some formatting issues in the .rst. Make sure to compile the rst to examine it; you can do this with github or with the |
Hey @wkerzendorf, I pushed the new index.rst file and created a pyplots directory where I put the plotting example. Waiting for feedback |
|
||
For more information about the Spectrum1d object go to `Spectrum 1D`_. | ||
|
||
Another common WCS is the **linear dispersion** and commonly serialized (encoded) to FITS keyword headers. For linear dispersion we are using the general `Spectrum1DPolynomialWCS`_ WCS. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be helpful to note somewhere which keywords are required for Spectrum1DPolynomialWCS to be successfully read. Is that in the docs anywhere you can link to?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spectrum1DPolynomialWCS
has no link to keywords it can be equally likely be initialized from a fit to an ascii table. The FITSHeader
class does that.
This looks very nice! |
Here you can see how your changes will look on readthedocs: http://specutils.readthedocs.org/en/doc-testing/ |
Oh I see! I ran make html in the docs and then opening the _build/html/index.html for a preview |
That does work as well - but this way we can see if it actually builds on readthedocs |
@keflavich merging |
@dinossimpson thanks for the update |
@wkerzendorf bitte schoen |
Hey @wkerzendorf, I made a summary from http://specutils.readthedocs.org/en/latest/specutils/index.html
Let me know if I should add or change something