-
Notifications
You must be signed in to change notification settings - Fork 54
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
Update for the linking documentation #210
Conversation
Codecov ReportBase: 41.01% // Head: 41.01% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## RC_v1.5.0 #210 +/- ##
==========================================
Coverage 41.01% 41.01%
==========================================
Files 14 14
Lines 2321 2321
==========================================
Hits 952 952
Misses 1369 1369
Flags with carried forward coverage won't be shown. Click here to find out more. Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
I think this is a great change; thanks for adding it.
I deleted a duplicate picture and corrected a typo. The updated page of the documentation can be found here: https://tobac--210.org.readthedocs.build/en/210/linking.html |
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.
Looks great! This is a really helpful contribution @snilsn, approved!
With two approving reviews, I'm happy to merge if you're happy for it to be merged @snilsn |
@freemansw1 After some feedback from @fsenf I increased some font sizes. Happy to merge now! |
I added some plots and explanations I originally did for my tutorial on the tracking methods to the linking documentation.