In core.reshape.pivot we have a couple places where a default kwarg is lib.NoDefault where the idiomatic usage would be lib.no_default. I assumed this was just a typo, but when I tried changing it, some tests broke. Need to track down what's going on there.