Update link margin analysis #364
Conversation
200km
added
priority::medium
|
@TomokiMochizuki こちら、major updateに当たると思いますので、#303 をベースに開発しPRもそちらに対して出してもらえると助かります。 |
200km
requested review from
200km,
seki-hiro,
sksat,
conjikidow,
suzuki-toshihir0 and
t-hosonuma
わかりました！ |
TomokiMochizuki
force-pushed
the
feature/update-antenna-pattern
branch
from
8e56266 to
4886243
Compare
TomokiMochizuki
changed the base branch from
develop
to
feature/major-update-v6.0.0
|
@200km |
|
同じシミュレーション条件でflagが立たなくなっている気がします． |
こちら再度確認してからまた連絡します． |
|
この問題は解決しました． |
|
@200km |
src/components/real/communication/ground_station_calculator.cpp
Outdated
Show resolved
Hide resolved
| @@ -14,6 +14,9 @@ is_receiver = 1 | |||
| // frequency [MHz] | |||
| frequency_MHz = 8200.0 | |||
|
|
|||
| // bitrate [bps] | |||
| tx_bitrate_bps = 10000 | |||
There was a problem hiding this comment.
[Q] bitrateはアンテナが決めるものではなく、その前段の送信機などで決めるものな気もするのですが、ここにこのパラメータを置くべきなのでしょうか？
There was a problem hiding this comment.
送信機などで決めるのであれば，tx_output_Wやfrequency_MHzもな気がしていて．どちらかに統一したかったです．
There was a problem hiding this comment.
frequency_MHzはアンテナも関わってくる気がするので、ここで良いのではないでしょうか？
tx_output_Wはたしかにそうなので、送信機側に動かしても良いと思います。
There was a problem hiding this comment.
frequency_MHzはアンテナも関わってくる気がするので、ここで良いのではないでしょうか？tx_output_Wはたしかにそうなので、送信機側に動かしても良いと思います。
それを行いますと，
tx_eirp_dBW_ = 10 * log10(tx_output_power_W_) + tx_parameters_.loss_feeder_dB_ + tx_parameters_.loss_pointing_dB_;の箇所がうまくいかなくなってしまうので，tx_eirp_dBWを別の場所に移行しなければならないので，アンテナの方に押し込んだほうがいいと思います．
There was a problem hiding this comment.
当初の設計からの問題ということですね。。。今の段階でひとまず動くようにしておく というのと、 より理想的な形に持っていく というのを両立するため、ひとまずここでは次のような対処にするというのはどうでしょう？
- 現時点では
AntennaクラスにSetTxBitrateやSetTxPowerなどの関数を準備して、これらの値を別の通信システムから設定できるようにしておく - 理想的な状態への改修に向けてTODOやFIXMEでコメントつけておく、issueも立てる
There was a problem hiding this comment.
- 現時点ではAntenna クラスにSetTxBitrateやSetTxPowerなどの関数を準備して、これらの値を別の通信システムから設定できるようにしておく
- 理想的な状態への改修に向けてTODOやFIXMEでコメントつけておく、issueも立てる
これに関してですが，現状送受信機のClassが存在していないので，とりあえずアンテナに帰属させるのがいいと思ったので，そのままにしています．GroundStationCalculatorにあるのも変なので．
そのうえで，FIXMEを切っておきました。
There was a problem hiding this comment.
- issueに関しては Rearrange Communication Components and Communication Calculation System #366 で切っておきました．
There was a problem hiding this comment.
Set関数を準備するというのは、「そのパラメータはひとまずアンテナに帰属するが、別のコンポが制御できるような余地を残しておく」という意図です。
Set関数を使わなかった時に使われるパラメータ初期値は今のような設定で良いので、動作としては今と特に変わらないかなと思います。
There was a problem hiding this comment.
とはいえ、Setは後から追加でも良い気もしてきましたし、issueも切ってくれたのでapproveしておきます
There was a problem hiding this comment.
ありがとうございます。
では、マージします
src/components/real/communication/ground_station_calculator.cpp
Outdated
Show resolved
Hide resolved
data/sample/initialize_files/components/ground_station_antenna.ini
Outdated
Show resolved
Hide resolved
TomokiMochizuki
force-pushed
the
feature/update-antenna-pattern
branch
from
6d9cdc8 to
1222b11
Compare
Overview
Update link margin analysis
Details
Validation results
compare the Python simulation result and S2E simulation result shown in the following figure.
Scope of influence
eg. The behavior of XX will be change
Supplement
Only for communication and ground station analysis.
Note
prioritylabel if possible.