Alejandro de la Vega

I spoke too soon as usually, obviously you can pass arguments in the JSON ``` j ={'roots': [{'transformer': 'FrameSamplingFilter', 'parameters': {'every': 10}, 'children': [{'transformer': 'BrightnessExtractor'}]}, {'transformer': 'STFTAudioExtractor'}]} ``` I think...

Amazon Rekognition - https://aws.amazon.com/rekognition/

Cool, I'll try to take a look this but if I don't feel free to bug me

Cool, okay two more things: 1) @rbroc can you take a quick look 2) We need to fix tests before we can merge! I'm guessing its unrelated but lets fix...

@anibalsolon I made a PR to your PR (so meta). LMK what you think. I think making the data type parameterize is still fine to do within this function bc...

Hey @anibalsolon Do you want to meet about this sometime soon? I'm happy to give this a full review. Also, can you rebase w/ master? I've fixed the tests recently

Tests are passing! I agree with @anibalsolon that the warning might be excessive to have every time the object is instantient. Perhaps we can catch a specific type of exception...

Test failures are unrelated, so merging

@rbroc do you remember if we fixed this? I feel like we did but I'm not seeing a PR for it

dlib is also super slow to install, which is a real drag.