scikit-spatial icon indicating copy to clipboard operation
scikit-spatial copied to clipboard

Added 2D Circle on plane in 3D space

Open Yeok-c opened this issue 1 year ago • 7 comments

Sphere's intersection on a 2D plane in 3D space is a circle on a plane. Test files and new class with barebone implementation added.

Yeok-c avatar Mar 06 '23 08:03 Yeok-c

Hi @Yeok-c,

Before I do a thorough review of this code, you need to get the PR checks (pre-commit and the tests) to pass. For starters, you need to delete all the commented-out code. For now I'm marking this PR as a draft.

Once these checks have passed, please mark this PR as ready for review and request me as a reviewer.

ajhynes7 avatar Mar 06 '23 12:03 ajhynes7

Hi @Yeok-c! Do you need help with this PR? If think this would be a very valuable addition to this library I have already developed something similar here, maybe it can be helpful.

CristianoPizzamiglio avatar Apr 10 '23 12:04 CristianoPizzamiglio

yeah I'd love some help. I've developed some more features on my fork https://github.com/Yeok-c/scikit-spatial

Circle3D object (a 2D circle on a plane in 3D space), resulting from sphere-plane intersection Elipse3D object (a 2D elipse on a plane in 3D space), resulting from cylinder-plane intersection

and also sphere-sphere-sphere trisection, which results in one points.

All the intersections are intentionally implemented without iterative algorithms, which makes it possible to do differentiable operations later.

The implementations are messy but they work for my purpose. I don't have time to align it with the style in this repo. I wouldn't mind if anyone added them to this repository with minor credits to me.

Yeok-c avatar Apr 11 '23 11:04 Yeok-c

yeah I'd love some help. I've developed some more features on my fork https://github.com/Yeok-c/scikit-spatial

Circle3D object (a 2D circle on a plane in 3D space), resulting from sphere-plane intersection Elipse3D object (a 2D elipse on a plane in 3D space), resulting from cylinder-plane intersection

and also sphere-sphere-sphere trisection, which results in one points.

All the intersections are intentionally implemented without iterative algorithms, which makes it possible to do differentiable operations later.

The implementations are messy but they work for my purpose. I don't have time to align it with the style in this repo. I wouldn't mind if anyone added them to this repository with minor credits to me.

Hi! Sorry, pretty busy at work. I will start working on this very soon!

CristianoPizzamiglio avatar May 07 '23 23:05 CristianoPizzamiglio

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

sonarqubecloud[bot] avatar Aug 18 '23 10:08 sonarqubecloud[bot]

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

sonarqubecloud[bot] avatar Apr 13 '24 03:04 sonarqubecloud[bot]

Quality Gate Failed Quality Gate failed

Failed conditions
3.4% Duplication on New Code (required ≤ 3%)
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

sonarqubecloud[bot] avatar Aug 17 '24 13:08 sonarqubecloud[bot]