search

carissalow / rapids

Reproducible Analysis Pipeline for Data Streams
http://www.rapids.science/
GNU Affero General Public License v3.0
36 stars 20 forks source link

Add data stream for AWARE Micro server #173

Closed shaabans closed 2 years ago

shaabans commented 2 years ago

Add support for the slightly different format of tables written by the AWARE Micro server (https://github.com/denzilferreira/aware-micro), where sensor data is contained within a data column in each table.

JulioV commented 2 years ago

Thanks a lot! It's great to see this was an easy change

shaabans commented 2 years ago

I hope we didn't miss anything, but we'll be heavily test driving this change over the next couple of weeks and I'll report back if more needs to be done :-) Thanks for merging!

JulioV commented 2 years ago

Sounds good! If you don't find any issues in the next couple of weeks can you let us know so we release it in a new version, please? Thanks!

CC @Meng6

shaabans commented 2 years ago

Will do, I'll set a reminder to slack you, and will definitely submit another PR if we run into any issues.

shaabans commented 2 years ago

@JulioV it seems like this has been working well after a week of testing. Email me if you'd like to join the test, which we'll be running for a few more weeks before the trial. If so, I can send you your RAPIDS results for a sniff test.

JulioV commented 2 years ago

Thanks Sam, that's ok we can release it, if we find any bugs we can always push a hot fix. Thanks a lot!

@Meng6 can you create the release whenever you have time please?