-
Notifications
You must be signed in to change notification settings - Fork 85
Conversation
Because I like it when people spell SparkFun correctly.
When a user wants to make a module configuration change they probably want the change to take effect immediately. The previous default of BBR didn't do a lot of good. I am also experimenting with generating key groups using defines.
Change Ublox to u-blox
Add support for TMOD3 fixed position.
Expand getVal/setVal method
Please don't merge until we get the library version rolled to 1.8.7 (separate PR - is there an easier way to get that in place?) |
I normally just update everything in the release_candidate branch, including updating library.properties (as you've just done), then merge into master and then issue a new release of master. I don't think there's an easier way of doing it? |
Update library.properties
Good point. Restrictions removed from release-candidate so we can be more flexible. No rush. Enjoy! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only two points:
One missing "b". Yes, really... (Just to prove I did look at the code!) :-D
And a question about the format of the keys. Should they be #defines or const uint32_t?
I've attached files that contain the full list in both formats. Please take your pick!
Adding AutoHPPOSLLH
I think we're closer to merge. I'm happy to leave this PR open until issue 131 is resolved, or we can merge without it. Up to you Paul. |
Update Example11_autoHPPOSLLH.ino
I think we're ready? I've just updated the example as Morten suggested. It's a lot better now. |
This PR: