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

Save center #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Save center #3

wants to merge 2 commits into from

Conversation

nytowl
Copy link

@nytowl nytowl commented Mar 3, 2013

Adds a centre calibration to the eeprom to reduce trimming

@DaneGardner
Copy link
Owner

I'm sorry it's taken so long to get back to you on this. I've been away on travel, and it's been nothing but rainy since I've been back, so I haven't been able to field test your changes.

However, after reviewing the code, I have one question. Why are you collecting the center of the collective value, but not using it? Is this something you tried to get working? or did you have future plans for it?

@nytowl
Copy link
Author

nytowl commented Mar 13, 2013

No problem, I've got a foot of snow in my yard so I haven't been able to do any testing lately either.

I wasn't able to get that working. I was going fix the serial comms and then have another go at it. What I tried was the save center collector value minus 400 but it wouldn't arm after that.

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