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

Issue #5479 resolved, Algorithm Frustum of Cone and tested it #5481

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Anurag127001
Copy link

@Anurag127001 Anurag127001 commented Sep 30, 2024

  • 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 code is formatted with clang-format -i --style=file path/to/your/file.java

was facing error regarding pom.xml file -lossy-conversions, to fix it needed to make some type casting and cleanup of code in AliquotSum.java, Caesar.java, PerfectNumber.java classes respectively.

now new VolumeOfFrustumOfCone method created and all test cases passed successfully of VolumeTest class
@siriak
Copy link
Member

siriak commented Oct 2, 2024

Please resolve conflicts

Copy link

github-actions bot commented Nov 2, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contribution!

@github-actions github-actions bot added the stale label Nov 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants