Should set_application_registry work with GenericUnitRegistry objects?

[cometbeetle]

Right now, the ApplicationRegistry.set method (in pint.registry) requires that the new_registry parameter be an instance of UnitRegistry or LazyRegistry, and therefore does not allow instances of GenericUnitRegistry to be used:

def set(self, new_registry):
    """Set the new registry

    Parameters
    ----------
    new_registry : ApplicationRegistry or LazyRegistry or UnitRegistry
        The new registry.

    See Also
    --------
    set_application_registry
    """
    if isinstance(new_registry, type(self)):
        new_registry = new_registry.get()

    if not isinstance(new_registry, (LazyRegistry, UnitRegistry)):
        raise TypeError("Expected UnitRegistry; got %s" % type(new_registry))
    logger.debug(
        "Changing app registry from %r to %r.", self._registry, new_registry
    )
    self._registry = new_registry

I was wondering if this is the intended behavior, or if it would make sense to also allow GenericUnitRegistry objects to be set as the application registry. I'd like to be able to use a custom generic unit registry class (as described here in the docs) with set_application_registry in a manner similar to the following:

class MyRegistry(
    pint.registry.GenericUnitRegistry[MyCustomQuantity, MyCustomUnit]
):
    Quantity = MyCustomQuantity
    Unit = MyCustomUnit

reg = MyRegistry()
pint.set_application_registry(reg)

Currently, doing something like that just raises a TypeError because of the behavior of ApplicationRegistry.set. Perhaps ApplicationRegistry.set could be adjusted to also allow GenericUnitRegistry objects?