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

Add effect of radio wave propagation time to geometric distance calculation #739

Conversation

fukudakazuya
Copy link
Contributor

@fukudakazuya fukudakazuya commented Feb 1, 2025

Related issues

#600

Description

以下を幾何学的距離の計算に追加した.

  • 見かけの幾何学的距離(受信機に信号が届いた時刻における受信機とGNSS衛星との距離)から電波伝播時間を計算
  • 受信機に信号が届いた時刻から電波伝播時間を差し引いた時刻におけるGNSS衛星の位置を計算
  • その位置と受信機に信号が届いた時刻における受信機の距離を計算
  • サニャック効果による補正量を追加

また,gnss_receiver.iniにpseudorange_loggingを追加し,擬似距離計算のログ出力をするかどうか設定できるようにした(前回実装し忘れていた)

Test results

受信機に信号が届いた時刻における受信機とGNSS衛星との距離と,電波伝播時間とサニャック効果を考慮した擬似距離(ノイズを0としているので幾何学的距離に等しい)との差をプロットした.
スクリーンショット 2025-02-01 12 15 13

Impact

Describe the scope of influence of the changes, e.g., The behavior of feature ** changes.

Supplementary information

Provide any supplementary information.

@fukudakazuya fukudakazuya added priority::medium priority medium component component emulation major update incompatible API changes labels Feb 1, 2025
@fukudakazuya fukudakazuya self-assigned this Feb 1, 2025
@fukudakazuya fukudakazuya requested review from sksat and a team as code owners February 1, 2025 03:41
@fukudakazuya fukudakazuya requested review from 200km, ogoogo, Hiro-0110, seki-hiro, suzuki-toshihir0, t-hosonuma and conjikidow and removed request for a team February 1, 2025 03:41
@200km 200km added this to the Major update v9.0.0 milestone Feb 1, 2025
Copy link
Member

@200km 200km left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

実装ありがとうございます。コメントつけたので参考にしてください。

@fukudakazuya
Copy link
Contributor Author

コメントありがとうございました.修正しました.
EpochTime型の誤りを修正し,受信機に信号が届いた時刻における受信機とGNSS衛星との距離と,電波伝播時間とサニャック効果を考慮した擬似距離(ノイズを0としているので幾何学的距離に等しい)との差をプロットしたところ以下のようになり,差が小さくなりました.
スクリーンショット 2025-02-03 15 04 16

@200km 200km self-requested a review February 3, 2025 07:00
Copy link
Member

@200km 200km left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

追加コメントつけました、

@@ -32,6 +32,11 @@ EpochTime::EpochTime(const DateTime date_time) {
fraction_s_ = date_time.GetSecond() - (double)second;
}

EpochTime::EpochTime(const double time_with_fraction_s) {
time_s_ = static_cast<uint64_t>(std::floor(time_with_fraction_s));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[MUST] 負の値の場合、0を入れたEpochTimeを返すようにお願いします。

@fukudakazuya
Copy link
Contributor Author

修正しました

@200km 200km self-requested a review February 4, 2025 09:23
Copy link
Member

@200km 200km left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正ありがとうございます。Approveしました。

@fukudakazuya fukudakazuya merged commit 6b7fe3c into feature/gnss-receiver-update Feb 4, 2025
10 checks passed
@fukudakazuya fukudakazuya deleted the feature/gnss-receiver-radio-wave-propagation branch February 4, 2025 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component component emulation major update incompatible API changes priority::medium priority medium
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants