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

SphericalCoordinates: make query methods const #452

Merged
merged 1 commit into from
Jul 5, 2022

Conversation

adityapande-1995
Copy link
Contributor

@adityapande-1995 adityapande-1995 commented Jul 5, 2022

Signed-off-by: Aditya aditya050995@gmail.com

🦟 Bug fix

Summary

This PR changes SurfaceRadius, AxisEquatorial, AxisPolar, Flattening methods to const, similar to other existing methods Heading(), LatitudeReference(), etc.

Also adds comments to exclude deprecated distance function from code coverage.

Follow up to #434

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

Signed-off-by: Aditya <aditya050995@gmail.com>
@github-actions github-actions bot added the 🌱 garden Ignition Garden label Jul 5, 2022
@adityapande-1995 adityapande-1995 changed the title Make query methods const Make spherical coord query methods const Jul 5, 2022
@adityapande-1995 adityapande-1995 requested a review from jennuine July 5, 2022 21:17
@codecov
Copy link

codecov bot commented Jul 5, 2022

Codecov Report

Merging #452 (bb46c61) into main (33a4827) will increase coverage by 0.03%.
The diff coverage is 100.00%.

❗ Current head bb46c61 differs from pull request most recent head f940826. Consider uploading reports for the commit f940826 to get more accurate results

@@            Coverage Diff             @@
##             main     #452      +/-   ##
==========================================
+ Coverage   99.65%   99.68%   +0.03%     
==========================================
  Files          73       73              
  Lines        6641     6639       -2     
==========================================
  Hits         6618     6618              
+ Misses         23       21       -2     
Impacted Files Coverage Δ
src/SphericalCoordinates.cc 99.68% <100.00%> (+0.63%) ⬆️

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 33a4827...f940826. Read the comment docs.

@scpeters scpeters changed the title Make spherical coord query methods const SphericalCoordinates: make query methods const Jul 5, 2022
@adityapande-1995 adityapande-1995 merged commit 3827f4c into main Jul 5, 2022
@adityapande-1995 adityapande-1995 deleted the aditya/const_methods branch July 5, 2022 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌱 garden Ignition Garden
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants