Skip to content

Implement MEASX message support #169

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

Closed
wants to merge 1 commit into from

Conversation

FerdinandvHagen
Copy link

Implements MEASX message support according to the Adding_New_Messages.md document.

Implements MEASX message support according to the Adding_New_Messages.md document.
@PaulZC
Copy link
Collaborator

PaulZC commented Nov 28, 2022

Thank you Ferdinand (@FerdinandvHagen ),

I will merge and test this in a few days - I am busy with some other developments at the moment.

Best wishes,
Paul

@PaulZC
Copy link
Collaborator

PaulZC commented Nov 28, 2022

Hi Ferdinand (@FerdinandvHagen ),

Unfortunately this is the straw that breaks the camel's back...

We have - for some time - been very close to exceeding the program memory on the ATmega328P (Uno / original RedBoard). Your changes take us over the limit...:

Sketch uses 32858 bytes (101%) of program storage space. Maximum is 32256 bytes.

So, with regret, I'm not going to merge your changes at the moment. I'll need to figure out a way to further reduce the program memory use first.

But thanks again for submitting this PR - much appreciated.

All the best,
Paul

This was referenced Dec 4, 2022
PaulZC added a commit that referenced this pull request Dec 18, 2022
@PaulZC
Copy link
Collaborator

PaulZC commented Dec 18, 2022

Hi Ferdinand (@FerdinandvHagen ),

I've added your code into the v3 branch. If you have time, please give it a try. I haven't tested it yet, but it compiles OK.

Best wishes,
Paul

@PaulZC
Copy link
Collaborator

PaulZC commented Jan 5, 2023

Hi Ferdinand (@FerdinandvHagen ),

I've made the v3 library public:

https://github.com/sparkfun/SparkFun_u-blox_GNSS_v3

It is still Work-In-Progress. I'm slowly working my way through the examples, updating and testing each one as I go. But I'm confident the library is useable as-is.

Please give it a try and let me know if you spot any gremlins. Thanks!

You won't be able to install it via the Arduino Library Manager. For now, you will need to download it and install it manually.

Thanks for the help. Let me know what you find. If you do find a problem, please open an Issue here:

https://github.com/sparkfun/SparkFun_u-blox_GNSS_v3/issues

All the best,
Paul

@PaulZC
Copy link
Collaborator

PaulZC commented Jan 10, 2023

Closing - thanks! Please see:

https://github.com/sparkfun/SparkFun_u-blox_GNSS_v3/releases/tag/v3.0.0

@PaulZC PaulZC closed this Jan 10, 2023
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.

2 participants