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

Sensibility needs a Python JSON parser #7

Closed
yyzhuang opened this issue Nov 20, 2014 · 3 comments
Closed

Sensibility needs a Python JSON parser #7

yyzhuang opened this issue Nov 20, 2014 · 3 comments

Comments

@yyzhuang
Copy link
Contributor

Currently we import lots of built ins in repy.py. We need to code our own or reuse other people's Python JSON parser to process returned sensor data.

@aaaaalbert
Copy link
Contributor

The current fix is that Python's json lib is imported in repy.py, and made accessible through a new, SensibilityTestbed-specific Repy API call.

(I think that in the future, the actual sensor calls will become part of the SensibilityTestbed Repy API.)

@JustinCappos
Copy link

I would be fine with the sensor calls being implemented in a security layer
instead. The repy sandbox could just let you open the socket to the
appropriate port, etc. The layer could actually implement the calls.

On Mon, May 18, 2015 at 10:47 AM, aaaaalbert notifications@github.com
wrote:

The current fix is that Python's json lib is imported in repy.py, and
made accessible through a new, SensibilityTestbed-specific Repy API call.

(I think that in the future, the actual sensor calls will become part of
the SensibilityTestbed Repy API.)


Reply to this email directly or view it on GitHub
#7 (comment)
.

@aaaaalbert
Copy link
Contributor

@JustinCappos, a usability side note: Unless implemented via SeattleTestbed/nodemanager#89 / SeattleTestbed/repy_v2#3, using security layers for adding sensors isn't a very user-friendly way to add calls to the API, as witnessed by our GPS blurring experiments:

start dylink.r2py encasementlib.r2py sensor_layer.r2py blur_wifi.r2py blur_location2city.r2py

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

No branches or pull requests

3 participants