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

Fix for loss of user-defined settings when calling begin() #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nuhash
Copy link

@nuhash nuhash commented Dec 15, 2020

Moved initilisation to constructor so that user defined settings are preserved when calling begin()

Made to address #27

Moved initilisation to constructor so that user defined settings are preserved when calling begin()
@nuhash nuhash changed the title Init fix Fix for loss of user-defined settings when calling begin() Dec 15, 2020
CAP1Sup added a commit to CAP1Sup/Arduino_LSM9DS1 that referenced this pull request Apr 30, 2022
CAP1Sup added a commit to CAP1Sup/Arduino_LSM9DS1 that referenced this pull request Apr 30, 2022
@CAP1Sup
Copy link

CAP1Sup commented Apr 30, 2022

I created my own fork and merged your changes into it. Unfortunately, I don't think that Sparkfun is going to continue to update this repo. I'll do my best to keep mine up to date. I know that I could just use your branch, but there are other issues, such as the flaw in the temperature reading processing

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