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

REDIS_URL should be a CL arg for sensors-src (or something) #7

Open
rpj opened this issue Dec 28, 2018 · 1 comment
Open

REDIS_URL should be a CL arg for sensors-src (or something) #7

rpj opened this issue Dec 28, 2018 · 1 comment
Assignees
Labels
code-smells-bad Not exactly a bug (things still work), but not ideal P2 Priority 2

Comments

@rpj
Copy link
Owner

rpj commented Dec 28, 2018

REDIS_URL handling needs to be fixed, is really the point.

@rpj rpj added the bug Something isn't working label Dec 28, 2018
@rpj rpj self-assigned this Dec 28, 2018
@rpj rpj added code-smells-bad Not exactly a bug (things still work), but not ideal P1 Priority 1 P2 Priority 2 and removed bug Something isn't working P1 Priority 1 labels Dec 31, 2018
@mheirendt
Copy link

I've configured redis in the config.json file as follows:
{ redis_config: { host: 'host', port: port, db: 0 } }

And am seeing the following stack trace:

Traceback (most recent call last): File "./sensors-src", line 45, in <module> ns = sens.create(s['name'], config=s['config'] if 'config' in s else None, redis_cfg=redis_cfg) File "/usr/projects/rpi/env/local/lib/python2.7/site-packages/rpjios/Sensors.py", line 29, in create return cf(**config) if config else cf() File "/usr/projects/rpi/env/local/lib/python2.7/site-packages/rpjios/SensorBase.py", line 17, in wrapper func(*args, **kwargs) File "/usr/projects/rpi/env/local/lib/python2.7/site-packages/rpjios/SensorBase.py", line 26, in wrapper func(*args, **kwargs) File "/usr/projects/rpi/env/local/lib/python2.7/site-packages/rpjios/sensors/SEN12642.py", line 9, in __init__ super(Factory, self).__init__(*args, **kwargs) File "/usr/projects/rpi/env/local/lib/python2.7/site-packages/rpjios/AnalogBase.py", line 10, in __init__ super(Analog, self).__init__(*args, **kwargs) File "/usr/projects/rpi/env/local/lib/python2.7/site-packages/rpjios/SensorBase.py", line 59, in __init__ self._sub = Subscriber(channel=self.channel) File "/usr/projects/rpi/env/local/lib/python2.7/site-packages/rpjios/SubscriberBase.py", line 17, in __init__ super(Subscriber, self).__init__(*args, **kwargs) File "/usr/projects/rpi/env/local/lib/python2.7/site-packages/rpjios/RedisBase.py", line 13, in __init__ self._r = redis.StrictRedis(**kwargs) TypeError: type object argument after ** must be a mapping, not NoneType Exception AttributeError: "'Subscriber' object has no attribute '_psunsubfunc'" in <bound method Subscriber.__del__ of <rpjios.SubscriberBase.Subscriber object at 0xb6519bd0>> ignored

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code-smells-bad Not exactly a bug (things still work), but not ideal P2 Priority 2
Projects
None yet
Development

No branches or pull requests

2 participants