Airspace-Encounter-Models / Airspace-Encounter-Models/em-model-manned-bayes

Altitude and speed bounds throw errors on default behavior with cor_v2p1

未关闭
#45 4 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
enhancement help wanted
主要语言
MATLAB
星标
22
派生
9
PR 合并指标
30 天内没有已合并 PR

描述

### Description
When running `sample2track.m` with a RADES correlated model (ex. `cor_v2p1.txt`), getters for altitude and speed bounds don't work properly. For altitude, getting `min_speed` and `max_alt` in lines 104-105 of `sample2.track.m` throws an exception because min/max alt bounds do not exist in `cor_v2p1.txt` at boundary index 2, the index of variable `L`. For speed, the default label (set in line 28) is `'v'`, which is not a valid label in the correlated model because there are only `'v_1','v_2'`.

### Reproducibility
`[isGood, ~] = sample2track(parameters_filename, initial_output_filename, transition_output_filename, ...
'out_dir_parent', out_dir_parent)
with valid inputs, and `parameters_filename = 'cor_v2p1.txt'`, should reproduce both issues.

### Expectation
I resolved this issue by simply adding altitude bounds back into `cor_v2p1.txt` consistent with the model report. For the speed bounds, I explicitly passed in `'label_initial_speed', 'v_1'` into the function call, but I think this should be fixed in such a way that the function does not break on default functionality (i.e. either this should be a required argument or it should know to use 'v' or 'v_1' depending on the model).

### Environment
Debian 10, MATLAB R2021a

贡献指南

这个仓库没有索引到贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。