google / google/s2-geometry-library-java

S1ChordAngleTest Failure

オープン
#44 コメント 9 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Java
スター
587
フォーク
231
PR マージ指標
30日以内にマージされた PR はありません

説明

I get the following failure when I run `mvn test`:

```
[ERROR] Tests run: 20, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.027 s <<< FAILURE! - in com.google.common.geometry.S1ChordAngleTest
[ERROR] com.google.common.geometry.S1ChordAngleTest.testTrigonometry Time elapsed: 0.003 s <<< FAILURE!
junit.framework.AssertionFailedError: expected:<0.15643446504023098> but was:<0.1564344650402278>
at junit.framework.Assert.fail(Assert.java:57)
...
```

Note that, as I mentioned in [issue 43](https://github.com/google/s2-geometry-library-java/issues/43), I had to change the `guava.version` from `32.0-jre` to `32.0.0-jre` in `pom.xml` to get `mvn compile` to succeed. That is the only local diff I have compared to the `master` branch.

In case it's relevant, when I run `java -version`, that returns

```
java version "22.0.1" 2024-04-16
Java(TM) SE Runtime Environment (build 22.0.1+8-16)
Java HotSpot(TM) 64-Bit Server VM (build 22.0.1+8-16, mixed mode, sharing)
```

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

S1ChordAngleTest.testTrigonometry から始め、Java 22 で mvn test を使用して失敗を再現します。Issue で報告されている guava.version の変更を含めて pom.xml を調査し、失敗している数値アサーションをプロジェクトの期待される動作と比較します。原因を特定して対処し、テストスイートが一貫して成功すれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
testing
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。