[python-abp] Let RPy recursively parse filter options to dicts

[rhowell]

Background

The data team (Stephan) requested this functionality.

What to change

In rpy.py, the tuple2dict function should recursively parse filter options to dicts.

Ex.
[("action", "block"), ("options", [("third-party", "true), ...])]

gets converted to

{"action": "block", "options": [("third-party", "true"), ...]}

instead of

{"action": "block", "options": {"third-party": "true", ...}}

i.e. the nested tuples don't get converted.

[rhowell]

I wasn't sure whether to mark this 'Ready'. Wanted to check first, if there's any reason not to implement it? Seems like a fairly small change that could help the data team. Does anyone else depend on rpy, that this change might effect?

[kvas]

python-abp should eventually get the functionality of representing the options in a more meaningful way and closer to how they are interpreted in ABP. It's already needed for example for the use case of ​abp-rcgen and I think the data team would also benefit from it. There's no spec for that yet and until it's implemented, this ticket is useful and it's quick so why not.

I'm happy to give guidance and feedback about better option API in python-abp. There might even be an implementation on top of an older version of it floating around somewhere. I just don't have time to do it myself right now.

[abpbot]

A commit referencing this issue has landed:
​Issue 7391 - Let rpy parse filter options to dicts