The above is fully PEP8 compliant. I usually only break after the opening ``(`` if one of the arguments is sufficiently long that it won't all fit inline below it.

Happy to change it, but we have this style elsewhere in the project.