sparkfun / Weather_Shield

Barometric pressure, temperature, humidity and light sensing weather shield for Arduino.
https://www.sparkfun.com/
Other
69 stars 67 forks source link

Memory optimizations #22

Closed ClintonKeith closed 8 years ago

ClintonKeith commented 9 years ago

I've integrated the weathershield with an Uno and Xbee using SoftSerial to communicate. Including the Xbee and SoftSerial libraries was a challenge due to the SRAM memory useage of the example sketch. I cleaned up a number of the memory usage problems and have others to go, but thought I'd raise the issue here for discussion before I create a pull request:

BTW: Really impressed with how quickly this shield, hardware and example came up and ran. Good job.

ToniCorinne commented 9 years ago

These all sounds like great improvements. Please do issue a pull request and we'll take a look at it. Even if it doesn't necessarily work for our examples, it's a great record to have out there for other users.

nseidle commented 8 years ago

Hi ClintonKeith - Yes please! If you get a good sketch put together and would like to share we'd be happy to cross link. I just touched up the hook-up guide and created some additional, more basic sketches to get beginners up and running. If you see places to improve the basic, weather station, or GPS sketches please do so and shoot us a pull request.

If I ever touch-up my weather stations I'll use that moment to circle back and implement some of your suggestions as well. I just can't make changes to the core code without a full setup to test it on.