Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/505 memcached documentation #509

Merged
merged 5 commits into from
Mar 8, 2024
Merged

Conversation

schroedk
Copy link
Collaborator

@schroedk schroedk commented Mar 7, 2024

Description

This PR closes #505

Changes

  • Raising specific error message with install instruction, when trying to load pydvl.utils.cache.memcached
    without pymemcache installed. If pymemcache is available, all symbols from
    pydvl.utils.cache.memcached are available through `pydvl.utils.cache

Checklist

  • Wrote Unit tests (if necessary)
  • Updated Documentation (if necessary)
  • Updated Changelog
  • If notebooks were added/changed, added boilerplate cells are tagged with "tags": ["hide"] or "tags": ["hide-input"]

Kristof Schroeder added 4 commits March 7, 2024 09:56
…stedLazyChunkSequence

to also include zarr stores, modify test to not write to disk anymore but to a zarr.MemoryStore
…ith a

hint on installation when module is loaded without pymemcach installed
@schroedk schroedk marked this pull request as ready for review March 7, 2024 10:03
@schroedk schroedk merged commit 1594431 into develop Mar 8, 2024
18 checks passed
@schroedk schroedk deleted the fix/505-memcached-documentation branch March 11, 2024 11:42
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.

NameError: name 'PickleSerde' is not defined
2 participants