Dimitri Papadopoulos Orfanos

Results 835 comments of Dimitri Papadopoulos Orfanos

About `deserialize_meta` in [plugins/plugin_utils.h](https://github.com/Blosc/c-blosc2/blob/main/plugins/plugin_utils.h) / [plugins/plugin_utils.c](https://github.com/Blosc/c-blosc2/blob/main/plugins/plugin_utils.c): 1. There's an inline `b2nd_deserialize_meta` function in [`include/b2nd.h`](https://github.com/Blosc/c-blosc2/blob/4ff59716e1c140c49b7aa688894ca0a20ea7e0ac/include/b2nd.h#L513-L599), which is almost identical to `deserialize_meta`. It is currently used in the library only, but its...

Indeed `CGLAGS` seems to be the traditional standard approach to modify C options locally outside the build chain, but also inside the build chain. For example, in `configure.ac` you would...

See also: * [CMake 3.10 and above already honors CFLAGS env-var](https://github.com/libsdl-org/SDL/pull/4681) * [[Patch] Don't override the default CMAKE_C_FLAGS](https://github.com/libsdl-org/SDL/issues/1819)

> I think a better type for these functions would be: > > ```python > async def list(self) -> AsyncIterator[str]: > ``` The roadmap defines the [Store API](https://github.com/zarr-developers/zarr-python/blob/main/docs/roadmap.rst#store-api) as follows...

Meanwhile, I tried to revert the return type from `AsyncGenerator[str, None]` to `AsyncGenerator[str]`, to see what happens. As for the `AsyncGenerator` → `AsyncIterator` change, how about addressing it in an...

OK, let me try then. It's just that's I'll have to modify all child classes.

I tried briefly, but I cannot get typing to work after rebasing. I need to take a thorough look.

[How to correctly specify type hints with AsyncGenerator and AsyncContextManager](https://stackoverflow.com/questions/68905848/how-to-correctly-specify-type-hints-with-asyncgenerator-and-asynccontextmanager)

See also https://github.com/pypa/distutils/issues/251, since [class `Command`](https://github.com/pypa/distutils/blob/e5b06e144d1e57e0efebbc943bf071cd22e46da8/distutils/cmd.py#L17) is defined in [distutils](https://github.com/pypa/distutils/).

See https://github.com/pypa/setuptools/issues/4421.