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

basic quickstart examples for spectral extraction #142

Merged
merged 2 commits into from
Oct 3, 2022

Conversation

kecnry
Copy link
Member

@kecnry kecnry commented Sep 27, 2022

This PR adds a very basic walkthrough of the steps and syntax involved for a spectral extraction workflow. It also moves the "Data reduction process" section to the bottom in a new "Development Docs" section.

Rendered Docs

@codecov
Copy link

codecov bot commented Sep 27, 2022

Codecov Report

Merging #142 (478c99c) into main (78e68b7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #142   +/-   ##
=======================================
  Coverage   70.57%   70.57%           
=======================================
  Files           9        9           
  Lines         605      605           
=======================================
  Hits          427      427           
  Misses        178      178           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@kecnry kecnry force-pushed the docs-quickstart branch 3 times, most recently from b09b439 to f18176b Compare September 27, 2022 19:37
@kecnry kecnry marked this pull request as ready for review September 27, 2022 19:46
Copy link

@PatrickOgle PatrickOgle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and hits the right tone and level of detail for a Quick Start.

Copy link
Contributor

@ojustino ojustino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will be a good contribution to the documentation. I have a couple of suggestions before we finalize it.

@kecnry kecnry requested a review from ojustino September 30, 2022 19:12
@kecnry kecnry merged commit f6a834f into astropy:main Oct 3, 2022
@kecnry kecnry deleted the docs-quickstart branch October 3, 2022 14:09
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

Successfully merging this pull request may close these issues.

3 participants