Skip to content

Comments

Correct memory overflow on getEEPROM#5

Open
Alfystar wants to merge 1 commit intoSPEC-Sensors:masterfrom
Alfystar:patch-1
Open

Correct memory overflow on getEEPROM#5
Alfystar wants to merge 1 commit intoSPEC-Sensors:masterfrom
Alfystar:patch-1

Conversation

@Alfystar
Copy link

To avoi problem during save and to semplificate same thing in the library

To avoi problem during save and to semplificate same thing in the library
Copy link
Collaborator

@dpeaslee dpeaslee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't get a chance to test, but it looks ok. Was using larger index values of counter to debug, but can now set to zeros.

@Alfystar
Copy link
Author

The modification are because the array are by 13 element, but the for count from 1 to 13 index value, in other words 2° element to the 14° element! (for the first for)
The second one is just for efficiency
Happy to help 😊

@sayedabbas722
Copy link

Hello

Why this library is not available in the Platformio library section

@dpeaslee
Copy link
Collaborator

dpeaslee commented Jun 14, 2021

Hello

Why this library is not available in the Platformio library section

I think I understand what you are asking, "why this hasn't been written for uC IDE's like PlatformIO"
First, let me explain that the libraries here are for developers who have easy access to Arduino.
The .cpp and .h files can be easily ported to other IDE's, but I suggest if you are developing using a novel uC to consider using your own circuit to implement.
#1 The ULPSM libraries are for Analog use of our open source potentiostat circuit.
#2 The DGS libraries are for Digital use of our UART enabled opensource digital sensor module

If you are using a uC with a high bit count ADC (12-16 bit ADC), I recommend using our ULPSM or equivalent circuit, either plugged or embedded into your device. You will probably want to build your own library to handle the sensor.

If you are using a uC with multiple UART inputs, then you can use our digital sensor module (or if you only need one sensor and have one UART channel). But for lowest power/cost, I would recommend you study our circuit and imbed into yours, as this module uses a microcontroller that may be redundant to yours.

@Alfystar
Copy link
Author

hi, why this pull steel open after so much time?

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.

3 participants