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

Expand docs #107

Merged
merged 11 commits into from
Jan 21, 2022
Merged

Expand docs #107

merged 11 commits into from
Jan 21, 2022

Conversation

e-koch
Copy link
Collaborator

@e-koch e-koch commented Jan 21, 2022

Update and expand the docs. And fix a few minor issues in the docs build.

@e-koch e-koch requested a review from keflavich January 21, 2022 18:55
@codecov-commenter
Copy link

codecov-commenter commented Jan 21, 2022

Codecov Report

Merging #107 (d27763d) into master (3332cba) will increase coverage by 0.14%.
The diff coverage is n/a.

❗ Current head d27763d differs from pull request most recent head 65ad730. Consider uploading reports for the commit 65ad730 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #107      +/-   ##
==========================================
+ Coverage   86.00%   86.15%   +0.14%     
==========================================
  Files          12       12              
  Lines        1365     1365              
==========================================
+ Hits         1174     1176       +2     
+ Misses        191      189       -2     
Impacted Files Coverage Δ
radio_beam/beam.py 85.65% <0.00%> (+0.77%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3332cba...65ad730. Read the comment docs.

docs/index.rst Outdated Show resolved Hide resolved
docs/index.rst Outdated Show resolved Hide resolved
docs/index.rst Outdated Show resolved Hide resolved
In the above example, the second FITS extension contains the beam tables, while the
first would have the spectral cube data.

To read a table of beams from a CASA image (must be run inside a CASA environment!)::
Copy link
Contributor

Choose a reason for hiding this comment

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

can we do this w/casa-formats-io now?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Probably. I raised an issue: radio-astro-tools/casa-formats-io#42.

@keflavich
Copy link
Contributor

merging as soon as CI passes

@e-koch
Copy link
Collaborator Author

e-koch commented Jan 21, 2022

@keflavich ready to merge.

@keflavich keflavich merged commit 9838af1 into radio-astro-tools:master Jan 21, 2022
@e-koch e-koch deleted the expand_docs branch January 21, 2022 22:14
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