Implement volume calculation for ellipsoid by kvadrik · Pull Request #7338 · TheAlgorithms/Java

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@kvadrik

Copy link Copy Markdown

Contributor

@kvadrik kvadrik commented

Mar 24, 2026

edited

Loading

Added a method to calculate the volume of an ellipsoid.

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized it.
  • All filenames are in PascalCase.
  • All functions and variable names follow Java naming conventions.
  • All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.
  • All new algorithms include a corresponding test class that validates their functionality.
  • All new code is formatted with clang-format -i --style=file path/to/your/file.java
Added a method to calculate the volume of an ellipsoid.

@codecov-commenter

Copy link Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.47%. Comparing base (76c45c1) to head (8f886cc).

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #7338   +/-   ##
=========================================
  Coverage     79.46%   79.47%           
- Complexity     7081     7084    +3     
=========================================
  Files           790      790           
  Lines         23160    23161    +1     
  Branches       4556     4556           
=========================================
+ Hits          18404    18407    +3     
  Misses         4021     4021           
+ Partials        735      733    -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@DenizAltunkapan DenizAltunkapan merged commit 010c955 into TheAlgorithms:master

Mar 25, 2026

7 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DenizAltunkapan DenizAltunkapan DenizAltunkapan approved these changes

@yanglbme yanglbme Awaiting requested review from yanglbme yanglbme is a code owner

@alxkm alxkm Awaiting requested review from alxkm alxkm is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kvadrik @codecov-commenter @DenizAltunkapan