Possible bug in compareEdges
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 62/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Bereich
- computer-graphics
Rechercherichtung
Beginne mit compareEdges in library/src/com/google/common/geometry/S2EdgeUtil.java ungefähr bei Zeile 3101 und vergleiche es anschließend mit src/s2/s2edge_crossings.h ungefähr bei Zeile 63 in der referenzierten C++-Implementierung. Verfolge die umgebenden Argumente und vorhandenen Tests, um zu bestimmen, welcher Vergleich beabsichtigt ist; abgeschlossen ist die Aufgabe, wenn sich das Verhalten von Java mit der verifizierten Implementierung deckt und die Tests erfolgreich durchlaufen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
The C++ and Java versions of compareEdges don't match as the C++ compare a1 and b1, while the Java compare b0 and b1.
I guess the C++ version is the correct one?
- Vorherrschende Sprache
- Java
- Sterne
- 588
- Forks
- 230
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus google/s2-geometry-library-java
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 68/100
google/s2-geometry-library-java#47 · 2 Kommentare ·
-
Added new ant target docs: "Generate documentation"Evtl. vergeben @ThomasThelen hat das vor 209 Tagen übernommen. Offenauto-migrated Type-Patch
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
google/s2-geometry-library-java#2 · 1 Kommentar ·
-
FR: Add bazel supportOffen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
-
S1ChordAngleTest FailureOffen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
google/s2-geometry-library-java#44 · 9 Kommentare ·
Alle Issues in google/s2-geometry-library-java
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
a2aproject/a2a-java#1275 ·
Maintainer antworten meist innerhalb von 1 Tag
-
module/ftp Type/Bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
ballerina-platform/ballerina-library#9476 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
anwilli5/coin-collection-android-US#515 ·
Maintainer antworten meist innerhalb von 1 Tag
-
status: waiting-for-triage type: bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
spring-projects/spring-integration#11654 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 Unter einer Stunde Anfängerfreundlichkeit 70/100
Vakore/ZappierGames#248 ·