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

Minor JSON fixes #110

Merged
merged 8 commits into from
Apr 16, 2015
Merged

Minor JSON fixes #110

merged 8 commits into from
Apr 16, 2015

Conversation

mfine
Copy link
Contributor

@mfine mfine commented Apr 15, 2015

Improve the python documentation and add to_json and from_json methods to the base SBP object.

/cc @mookerji

@swiftnav-jenkins
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://zazu.ci.swiftnav.com/job/libsbp_Pull_Requests/30/
Test PASSed.

@swiftnav-jenkins
Copy link
Contributor

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://zazu.ci.swiftnav.com/job/libsbp_Pull_Requests/31/
Test FAILed.

@mfine
Copy link
Contributor Author

mfine commented Apr 15, 2015

please retest this

@mfine
Copy link
Contributor Author

mfine commented Apr 15, 2015

retest this please

@swiftnav-jenkins
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://zazu.ci.swiftnav.com/job/libsbp_Pull_Requests/32/
Test PASSed.

@swiftnav-jenkins
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://zazu.ci.swiftnav.com/job/libsbp_Pull_Requests/37/
Test PASSed.

@swiftnav-jenkins
Copy link
Contributor

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://zazu.ci.swiftnav.com/job/libsbp_Pull_Requests/38/
Test FAILed.

@swiftnav-jenkins
Copy link
Contributor

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://zazu.ci.swiftnav.com/job/libsbp_Pull_Requests/39/
Test FAILed.

@swiftnav-jenkins
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://zazu.ci.swiftnav.com/job/libsbp_Pull_Requests/40/
Test PASSed.

@swiftnav-jenkins
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://zazu.ci.swiftnav.com/job/libsbp_Pull_Requests/41/
Test PASSed.

@mfine
Copy link
Contributor Author

mfine commented Apr 15, 2015

This ended up being a catch all for a number of issues impacting the JSON log analysis. It does generate a new set of python files for changes in the generator. It's also skipping the pickle log tests for now.

/cc @fnoble

@mfine
Copy link
Contributor Author

mfine commented Apr 15, 2015

Towards supporting #110.

@mfine
Copy link
Contributor Author

mfine commented Apr 15, 2015

Ugh, towards supporting https://github.com/swift-nav/sbp_log_analysis/issues/97.

mookerji added a commit that referenced this pull request Apr 16, 2015
@mookerji mookerji merged commit 2d4bcf0 into swift-nav:master Apr 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants