-
Notifications
You must be signed in to change notification settings - Fork 25
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
adding details on how to citation the package and specific models #212
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #212 +/- ##
=======================================
Coverage 99.88% 99.88%
=======================================
Files 8 8
Lines 902 902
=======================================
Hits 901 901
Misses 1 1 ☔ View full report in Codecov by Sentry. |
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.
Just 2 small comments.
Otherwise, it looks great.
Thanks for this update!
One additional comment/question. What is the purpose of the reference API: https://dust-extinction.readthedocs.io/en/stable/dust_extinction/reference_api.html? It looks like some of that page is broken. |
This page provides the programming interface info. There are supposed to be graphical flow diagrams. Graphviz is clearly not working. Will have to investigate. Not part of this PR. Will open an issue. |
Closes #206
Closes #165