Skip to content

Conversation

@mfine
Copy link
Contributor

@mfine mfine commented Jun 26, 2015

Little tweaks to make the doc work. Find the generated doc here:

https://github.com/mfine/libsbp/blob/mfine-doc-tweaks/docs/sbp.pdf

/cc @cbeighley @denniszollo @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/163/
Test PASSed.

@mookerji
Copy link
Contributor

Man, I could swear we did something to break up that message ID table across two pages.

@mfine
Copy link
Contributor Author

mfine commented Jun 26, 2015

yeah, looks like we've got too many messages now

@mookerji
Copy link
Contributor

We should limit our message table to only those that can fit on a single A4 sheet of paper.

@mookerji
Copy link
Contributor

Single page is fine, we can fuck with it later.

@mookerji
Copy link
Contributor

Add the semantic versioning policy as well?
https://github.com/mfine/libsbp/blob/mfine-versioning/VERSIONING.md

@mfine
Copy link
Contributor Author

mfine commented Jun 26, 2015

Doc driven development

@mfine
Copy link
Contributor Author

mfine commented Jun 26, 2015

Yeah, we'll add the versioning policy in when its ready - we're just at point release here.

@mookerji
Copy link
Contributor

👍 merge at will.

mfine added a commit that referenced this pull request Jun 26, 2015
@mfine mfine merged commit 803d847 into swift-nav:master Jun 26, 2015
@mfine
Copy link
Contributor Author

mfine commented Jun 26, 2015

Also, that cut line I put in there is super non-portable.. I tried sed, but it puked - will need to revisit.

@drewshannon
Copy link

Minor grammatical errors:

  • Section 2 "Message Framing Structure": First sentence, extra space between 0.46 and comma ("As of Version 0.46 , the frame consists...") - perhaps intentional?
  • Section 5 "Stable Message Definitions": First sentence, change "reporting" to "report"
  • Page 34: Two periods at the end of third sentence
  • General throughout (minor): inconsistent spacing between sentences.
  • Page 42, second sentence: word "be" is typed twice ("...but can also be be host access...")

RReichert pushed a commit that referenced this pull request Oct 7, 2025
…rgets (#187) (#1510)

Automated PR by Jenkins. If CI has passed successfully, merge away!



**cmake**  61cdba17  ->  1ab487a2
- 1ab487a2 : Default C++/C version for all swift targets
(swift-nav/cmake#187)


This pull request was created by
https://jenkins.ci.swift-nav.com/job/CI%20Infra/job/submodule-update/21466/.
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.

4 participants