Skip to content
This repository has been archived by the owner on Oct 1, 2023. It is now read-only.

Don't crash on JSON load if datasets are missing #443

Closed
calcmogul opened this issue Sep 8, 2022 · 0 comments · Fixed by #526
Closed

Don't crash on JSON load if datasets are missing #443

calcmogul opened this issue Sep 8, 2022 · 0 comments · Fixed by #526
Labels
component: analysis Data analysis (feedforward and feedback)

Comments

@calcmogul
Copy link
Member

Reported by https://www.chiefdelphi.com/t/sysid-crashes-whilst-opening-previously-saved-json-data/414812/

Could be this for loop.

for (auto&& key : AnalysisManager::kJsonDataKeys) {
data[key] = m_json.at(key).get<std::vector<Data>>();
}

@calcmogul calcmogul added the component: analysis Data analysis (feedforward and feedback) label Sep 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: analysis Data analysis (feedforward and feedback)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant