We should at least make this argument name more PEP 8-ish with an underscore: `s_id`. However, I'd *much* prefer it without the abbreviation, it took me a minute to realize what this argument was supposed to be when reading the code. How about `service_id` (my preference), `service_type`, or just `service`?