-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Export OD results in Parquet #147
Export OD results in Parquet #147
Conversation
+ Fix semi minor axis computation (was NaN for non hyperbolic orbits) + Add a `timestamped` initializer for ExportCfg (and related TODOs) + Remove heterogeneous `rms_pre/postfit_residual` function, use `rms_residual_ratios` instead + Remove old code in `trigger.rs` Signed-off-by: Christopher Rabotin <[email protected]>
Signed-off-by: Christopher Rabotin <[email protected]>
Signed-off-by: Christopher Rabotin <[email protected]>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #147 +/- ##
==========================================
+ Coverage 64.92% 66.40% +1.48%
==========================================
Files 126 127 +1
Lines 33087 32368 -719
==========================================
+ Hits 21482 21495 +13
+ Misses 11605 10873 -732
☔ View full report in Codecov by Sentry. |
Signed-off-by: Christopher Rabotin <[email protected]>
Cleaning up tasks:
|
Signed-off-by: Christopher Rabotin <[email protected]>
…arquet Signed-off-by: Christopher Rabotin <[email protected]>
Signed-off-by: Christopher Rabotin <[email protected]>
Signed-off-by: Christopher Rabotin <[email protected]>
Signed-off-by: Christopher Rabotin <[email protected]>
Add pure predictor test
Remove StateFormatter and NavSolutionFormatter, use `to_parquet` instead. Signed-off-by: Christopher Rabotin <[email protected]>
One last thing to make this immediately usable:
|
These are no longer relevant given the switch to parquet export Signed-off-by: Christopher Rabotin <[email protected]>
the csv crate is still needed for the Cosmo export Signed-off-by: Christopher Rabotin <[email protected]>
Signed-off-by: Christopher Rabotin <[email protected]>
Signed-off-by: Christopher Rabotin <[email protected]>
cargo +nightly udeps Signed-off-by: Christopher Rabotin <[email protected]>
Signed-off-by: Christopher Rabotin <[email protected]>
timestamped
initializer for ExportCfg (and related TODOs)rms_pre/postfit_residual
function, userms_residual_ratios
insteadtrigger.rs
MergeCompiler got very unhappy with all permutations I tried of this. This also complicates the fact that Estimate would need to know what kind of measurement is used: that isn't a deal-breaker but it isn't necessarily the most evident idea. For example, this would make it impossible to share aResidual<T>
andEstimate
KfEstimate
between a filter that runs on Range and one that ingests RangeDoppler.Add Kalman gain toRequires the above.Estimate
(as an option since it isn't set for a time update)RemoveNow irrelevantpredicted: bool
from Estimate and change to apub const
function depending on whether there is a residual or notExportCfg
to Python interface to control how to export data in parquetEffects
Closes #134
If this is a new feature or a bug fix ...
issue-xyz
wherexyz
is the number of the issue.If this change adds or modifies a validation case