Skip to content
This repository was archived by the owner on Oct 14, 2023. It is now read-only.

coesa76.py table documentation isn't consistent with code #1518

Closed
reine249 opened this issue Jun 13, 2022 · 1 comment · Fixed by #1520
Closed

coesa76.py table documentation isn't consistent with code #1518

reine249 opened this issue Jun 13, 2022 · 1 comment · Fixed by #1520

Comments

@reine249
Copy link
Contributor

🐞 Problem

Line 23 of the coesa76.py file. The fourth entry in the "T (K)" column should be 228.65 (not 288.650). This is only a documentation typo fix.

📋 Steps to solve the problem

  • Comment below about what you've started working on.
  • Add, commit, push your changes
  • Submit a pull request and add this in comments - Addresses #<put issue number here>
  • Ask for a review in comments section of pull request
  • Celebrate your contribution to this project 🎉
@astrojuanlu
Copy link
Member

Thanks a lot @reine249 for pointing that out! For reference:

+--------+---------+---------+-----------+---------------+---------------+
| 32.162 | 32.0 | 288.650 | 8.6801e0 | 1.3225e-2 | 2.8 |
+--------+---------+---------+-----------+---------------+---------------+

Would you like to make the changes yourself? You can do so by going to the file and then clicking the ✏️ edit button on the top right.

reine249 added a commit to reine249/poliastro that referenced this issue Jun 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants