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

updated docstring readthedocs #267

Merged
merged 3 commits into from
Jan 5, 2024
Merged

updated docstring readthedocs #267

merged 3 commits into from
Jan 5, 2024

Conversation

abhi0395
Copy link
Member

@abhi0395 abhi0395 commented Jan 5, 2024

I just realized that the docstrings for certain functions were not formatted correctly, leading to unclean description on https://redrock.readthedocs.io/en/latest/api.html. I have modified the docstrings for the following functions to ensure proper formatting.

archetypes.nearest_neighbour_model()
zscan.per_camera_coeff_with_least_square_batch()

@abhi0395 abhi0395 requested a review from sbailey January 5, 2024 05:20
@coveralls
Copy link

coveralls commented Jan 5, 2024

Coverage Status

coverage: 33.193% (+0.09%) from 33.103%
when pulling 6330c58 on abhijeet_archetype_docstring
into 36ebd46 on main.

@sbailey sbailey merged commit f1a1d06 into main Jan 5, 2024
12 checks passed
@sbailey sbailey deleted the abhijeet_archetype_docstring branch January 5, 2024 21:01
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