James Braza

Results 182 comments of James Braza

I see, so basically one has to end in `.weights.h5`, and I should put indexes such as `[1]` before `.weights.h5`. > tensorflow.python.framework.errors_impl.NotFoundError: Unsuccessful TensorSliceReader constructor: Failed to find any matching...

Note I am using [this fork of `openopc`](https://github.com/mkwiatkowski/openopc), not `openopc2`, mainly because I couldn't get `openopc2` multithreading to work. I found installing Matrikon OPC Explorer onto the client machine installs...

Thanks for the speedy response! > Multithreading is supported via the gateway. This is good, it focuses us on the `OpcDaClient` then. > Just make sure you create the proxy...

@eliabieri a nit on the label applied is I think this a bug, not a question. I don't think it's possible to use `OpcDaClient` with multiple threads, try running the...

Hi @renzop thank you for responding! `_pyroClaimOwnership` is not a method in `OpcDaClient`, so I don't think your suggestion is valid. See the below: ```python from openopc2.config import OpenOpcConfig from...

Also, in this part of the `README.md`: > On your Linux machine > > ```shell > pip install openopc2 > ``` > > python > > ```python > from openopc2.da_client...

Thanks @gugarosa for finding a workaround, that works because `get_imports` includes a special regex for `try`-`except`: https://github.com/huggingface/transformers/blob/v4.36.2/src/transformers/dynamic_module_utils.py#L149. --- To share, adding the below case to https://github.com/huggingface/transformers/blob/v4.36.2/tests/utils/test_dynamic_module_utils.py will expose the issue:...

Note a generalized importer should also be able to take into account `contextlib.suppress`: ```python import contextlib with contextlib.suppress(ImportError): from flash_attn import flash_attn_func ```

I think a fix here would be useful @github-actions, so let's keep it open

Oh wow, thank you so much for the excellent answer! That totally clears me up on how to proceed. I still think it's unideal that the default `Graph` behavior is...