Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(skore-hub-project/artifact/pickle): Use report.id as checksum - #2762

Draft
thomass-dev wants to merge 20 commits into
mainfrom
skore-hub-project-use-report-id
Draft

feat(skore-hub-project/artifact/pickle): Use report.id as checksum#2762
thomass-dev wants to merge 20 commits into
mainfrom
skore-hub-project-use-report-id

Conversation

@thomass-dev

@thomass-dev thomass-dev commented Apr 14, 2026

Copy link
Copy Markdown
Collaborator

Extract from #2114.


  • Remove serializer.py
    Make compute and checksum two different functions in the Artifact class.
    It allows checksum to be independently calculated from the representation of the content to upload.
  • Use report.id as checksum of the pickle artifact instead of blake2b(joblib.dump(report))
  • Explicit and assert that an artifact is not null, computed and uploaded before being sent to the hub
  • Improve readability in tests

@thomass-dev thomass-dev changed the title feat(skore-hub-project/artifact/pickle): Use report.id as pickle identifier feat(skore-hub-project/artifact/pickle): Use report.id as checksum Apr 14, 2026
Comment thread skore-hub-project/src/skore_hub_project/artifact/pickle/pickle.py
@thomass-dev thomass-dev self-assigned this Apr 15, 2026
@thomass-dev
thomass-dev force-pushed the skore-hub-project-use-report-id branch 2 times, most recently from 306d60a to 759bc4d Compare April 15, 2026 15:36
@thomass-dev
thomass-dev force-pushed the skore-hub-project-use-report-id branch from 759bc4d to efb47b4 Compare April 16, 2026 12:07
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.

1 participant