LGTM, just one small note. I see there 4 times usage of p["name"]:"". It would be nice to have it separated like to own string variable.