Skip to content

Do not lock file by default through h5py.File arg - #133

Open
loichuder wants to merge 1 commit into
mainfrom
locking-arg
Open

loichuder wants to merge 1 commit into
mainfrom
locking-arg

Conversation

@loichuder

Copy link
Copy Markdown
Member

Fix #128

Comment thread src/h5grove/utils.py
h5py_options: dict[str, Any] = {},
) -> Iterator[h5py.File]:

merged_h5py_options = {**DEFAULT_H5PY_OPTIONS, **h5py_options}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By merging dict this way, h5py_options can override the default options.

@loichuder
loichuder requested a review from axelboc October 19, 2026 15:13
@loichuder loichuder changed the title Lock file by default through h5py.File arg Do not lock file by default through h5py.File arg Oct 23, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consider passing locking=false by default when creating h5py File

1 participant