search

sandeepmistry / node-sensortag

Node.js lib for the TI SensorTag
MIT License
218 stars 111 forks source link

setIrTemperaturePeriod Implementation #24

Closed Andy1978 closed 9 years ago

Andy1978 commented 9 years ago

In your README.md, you describe

sensorTag.setIrTemperaturePeriod(period, callback); // period min 300ms, default period is 1000 ms

but I can't find an implementation for this. When I explore my Sensortag, I doubt this one has a Period Setting for the IR sensor

sandeepmistry commented 9 years ago

@Andy1978 I haven't pushed this feature to npm yet, that's probably why you don't see it.

sandeepmistry commented 9 years ago

Just published v0.1.10 to npm.

Andy1978 commented 9 years ago

Okay, I see this is a firmware issue. My sensor tag is 1.4.1 and hasn't a handle for IRTemperaturePeriod. This is also the reason why there are different PDFs for the gatt attribute table out there.

sandeepmistry commented 9 years ago

@Andy1978 that makes more sense.

I'm running 1.5 (Oct 23 2013) on my Sensortag, update via the TI Sensortag iOS app.

Andy1978 commented 9 years ago

I don't use IOS or Android so it would be hard to update. Just want to inform you, that you should consider in your source code, that the handles or available characteristics may change with different firmware versions.