Possible bug in compareEdges
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 62/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
調査の方向性
library/src/com/google/common/geometry/S2EdgeUtil.java の 3101 行付近にある compareEdges から始め、次に参照されている C++ 実装の 63 行付近にある src/s2/s2edge_crossings.h と比較します。周辺の引数と既存のテストを追跡して、どの比較が意図されているかを判断します。完了とは、Java の動作が検証済みの実装と一致し、テストに合格することを意味します。
索引モデルが issue の本文から書いたものです。
説明
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?
- 主要言語
- Java
- スター
- 587
- フォーク
- 232
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
google/s2-geometry-library-java のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 68/100
google/s2-geometry-library-java#47 · コメント 2 件 ·
-
Added new ant target docs: "Generate documentation"対応中かも @ThomasThelen が 180 日前に担当しました。 オープンauto-migrated Type-Patch
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
google/s2-geometry-library-java#2 · コメント 1 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
google/s2-geometry-library-java#44 · コメント 9 件 ·
google/s2-geometry-library-java の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
apache/skywalking#14120 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
HMCL-dev/HMCL#6943 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
micronaut-projects/micronaut-core#13677 ·
メンテナーはふだん 1 日以内に返信
-
new feature
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
apache/rocketmq-dashboard#5594 ·
メンテナーはふだん 3 日以内に返信