Skip to content

v0.0.3 - #6

Merged
scpwais merged 121 commits into
masterfrom
v0.0.3
Aug 20, 2020
Merged

v0.0.3#6
scpwais merged 121 commits into
masterfrom
v0.0.3

Conversation

@scpwais

Copy link
Copy Markdown
Contributor

In this revision:

  • Major clean-up / re-structure of python protobag
  • Python and C++ examples in the examples subdir
  • A demo python notebook

@rreusser@Erkaman@dustindorroh@cczarnocki would love your feedback on the python notebook demo. For now it's easiest to just read the version stored on Github (versus try to run it). Here's a permalink to the latest revision at time of writing: https://github.com/StandardCyborg/protobag/blob/aed9ea01d91eaecc838972a0a85bfc4d79b0d6d3/examples/protobag-to-parquet/protobag-demo-full.ipynb

@scpwais

Copy link
Copy Markdown
ContributorAuthor

@Erkaman@rreusser@dustindorroh@cczarnocki

The python notebook now runs on Colab! Since the repo isn't open source, you have to do a few things to get it working:

Now you can go to Runtime > Run All to run all cells, or run them one by one, or whatever you want.

@Erkaman

Copy link
Copy Markdown
Contributor

"The user provides as parameters a 'max_slop' (a time duration) and a 'max_queue_size' (a number of messages to buffer"

buffer? what does this buffer do? its the first time its even mentioned.

@Erkaman

Erkaman commented Jul 31, 2020

Copy link
Copy Markdown
Contributor

"If topics have messages recorded at somewhat different times (e.g. two sensors both record at 5Hz, but one records 50ms before the other), then MaxSlopTimeSync will find the nearest matches."

ok. im interested in how it actually does this. how finds the nearest matches. please write down a short summary of the algorithm used. otherwise, the thing feels like a black-box to me, which i dont like,

@Erkaman

Copy link
Copy Markdown
Contributor

I can definitely see the utility of using MaxSlopTimeSync for syncing up depth and rgba data captured from the iphone sensors. 👍👍👍

@Erkaman

Copy link
Copy Markdown
Contributor

spec.set_max_slop(seconds=2, nanos=0)

what does the nanos=0 argument do?

@Erkaman

Copy link
Copy Markdown
Contributor

@scpwais left all my feedback above.

@scpwais

Copy link
Copy Markdown
ContributorAuthor

thanks @Erkaman ! Linked in what you read is the MaxSlopTimeSync docs; here's that link again: MaxSlopTimeSync I think that will answer most of your questions. You might also consult the unit tests for MaxSlopTimeSync and for IterProducts. I'll update the notebook a bit to make these more conspicuous.

@scpwais

Copy link
Copy Markdown
ContributorAuthor

"what does the nanos=0 argument do?" -- nanos is nanoseconds

@scpwais
scpwais merged commit 785c7ec into masterAug 20, 2020
@Erkaman
Erkaman deleted the v0.0.3 branch August 26, 2020 22:16
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@scpwais@Erkaman