I think adding a py.typed marker to the package before all public APIs are typed (even just with a whole bunch of Anys) wasn't ideal. Downstream users will now see a bunch of typing errors because type checkers now assume that pyarrow provides type hints that can be relied upon.
In my case, I'm seeing mypy complain about Module has no attribute "count" [attr-defined] where the module in question is pyarrow.compute.
Please consider either removing the marker until type hints are ready or to make sure all public APIs exist (at least as def xyz(*args: Any, **kwargs: Any) -> Any stubs). Or apply the same __getatrr__ placeholder to all modules, not just the top-level module.
Originally posted by @robsdedude in #48618 (comment)
I think adding a
py.typedmarker to the package before all public APIs are typed (even just with a whole bunch ofAnys) wasn't ideal. Downstream users will now see a bunch of typing errors because type checkers now assume that pyarrow provides type hints that can be relied upon.In my case, I'm seeing
mypycomplain aboutModule has no attribute "count" [attr-defined]where the module in question ispyarrow.compute.Please consider either removing the marker until type hints are ready or to make sure all public APIs exist (at least as
def xyz(*args: Any, **kwargs: Any) -> Anystubs). Or apply the same__getatrr__placeholder to all modules, not just the top-level module.Originally posted by @robsdedude in #48618 (comment)