search

timothybrown / BSEC-Conduit

A first class Systemd process which acts as a conduit between between BSEC Library and MQTT.
MIT License
13 stars 11 forks source link

NEW Library unsupported #6

Open badstraw360 opened 5 years ago

badstraw360 commented 5 years ago

Your software is amazing!! Is it possible to update the code to support the latest library please?

Bosch just released version v1.4.7.4, and now after upgrading I'm getting this error message:

Captura de Pantalla 2019-07-10 a la(s) 14 36 44

I would appreciate your help. :)

jonizen commented 5 years ago

The problem seems to originate due to folder changes in the lib.

in init.py we can see that arch is set here:

return 'Normal_version/RaspberryPI/PiZero_ArmV6-32bits' and later used with: build_command = [ 'cc', '-Wall', '-Wno-unused-but-set-variable', '-Wno-unused-variable', '-static', '-iquote{}/API'.format(src_dir), '-iquote{}/algo/bin/{}'.format(src_dir, lib_arch), '-iquote{}/examples'.format(src_dir), '{}/API/bme680.c'.format(src_dir), '{}/examples/bsec_integration.c'.format(src_dir), '{}/bsec-library.c'.format(src_dir), '-L{}/algo/bin/{}'.format(src_dir, lib_arch), '-lalgobsec', '-lm', '-lrt', '-o', exec_dst ]

This will give you a path that with this: '-iquote{}/algo/bin/{}'.format(src_dir, lib_arch),

so the path becomes /algo/bin/Normal_version/RaspberryPI/PiZero_ArmV6-32bits but the folder structure from new release is: BSEC_1.4.7.4_Generic_Release/algo/normal_version/bin/RaspberryPI/PiZero_ArmV6-32bits i am not sure if the capital N is a problem to, but the easiest fix would be to just remove /algo/bin and put /algo/ then where we set the arch just add bin after normal_version and put N to lower :)

timothybrown commented 5 years ago

I’ll take care of this sometime this week! Sorry for the delay in getting to this, I was out of town when you opened this issue and it slipped my mind when I got back!

badstraw360 commented 5 years ago

I hope you have time soon to update the library, this is the only software that I found which is useful for this sensor.

jonizen commented 5 years ago

I have fixed the code and are currently running it on my Pi, i can put the changes in a PR, so that you at least can make it work in wait for a fix :) it's not that much, I can push it tomorrow :)

badstraw360 commented 5 years ago

I have fixed the code and are currently running it on my Pi, i can put the changes in a PR, so that you at least can make it work in wait for a fix :) it's not that much, I can push it tomorrow :)

thanks man, I would appreciate that !!!!

timothybrown commented 5 years ago

Sorry guys, I got caught up with work and other projects. I have the new code working, I’ll clean it up and upload it today.

Sorry about the delay!

badstraw360 commented 5 years ago

Sorry guys, I got caught up with work and other projects. I have the new code working, I’ll clean it up and upload it today.

Sorry about the delay!

Hey Timothy! I hope you can give us a hand to update the software, I think it’s really amazing how good it works, it’s truly very helpful.

badstraw360 commented 4 years ago

Would you update the software please??? It doesn't work anymore with the latest library..

PLEEEEEASE ...

m4dm4xi commented 4 years ago

I´d also be very happy about the updated software!

jonizen commented 4 years ago

I forked the repo currently running the updated code. Have a look at the changes or use the fork 😊 @m4

https://github.com/jonizen/BSEC-Conduit

I´d also be very happy about the updated software!

Would you update the software please??? It doesn't work anymore with the latest library..

PLEEEEEASE ...

c64emulator commented 4 years ago

In the meantime BOSCH went forward with its library to version 1.4.8.0.