Skip to content

fix: improve performance in dataset loading#607

Merged
guilhem-barthes merged 10 commits into
mainfrom
fix/improve-dataset-loading
Mar 24, 2023
Merged

fix: improve performance in dataset loading#607
guilhem-barthes merged 10 commits into
mainfrom
fix/improve-dataset-loading

Conversation

@guilhem-barthes
Copy link
Copy Markdown
Contributor

@guilhem-barthes guilhem-barthes commented Mar 1, 2023

Description

How has this been tested?

Checklist

  • changelog was updated with notable changes
  • documentation was updated

@github-actions github-actions Bot added the api label Mar 1, 2023
@guilhem-barthes
Copy link
Copy Markdown
Contributor Author

/e2e --tests sdk,frontend,substrafl --benchmarks mnist,camelyon

@Owlfred
Copy link
Copy Markdown

Owlfred commented Mar 1, 2023

End to end tests: ✔️ SUCCESS

@guilhem-barthes guilhem-barthes force-pushed the fix/improve-dataset-loading branch from 3e9588d to 690bfcb Compare March 13, 2023 08:38
Comment thread backend/api/models/datamanager.py Outdated
@guilhem-barthes guilhem-barthes force-pushed the fix/improve-dataset-loading branch from 690bfcb to 1248fbf Compare March 24, 2023 14:26
@guilhem-barthes guilhem-barthes marked this pull request as ready for review March 24, 2023 15:10
@guilhem-barthes guilhem-barthes requested a review from a team as a code owner March 24, 2023 15:10
Signed-off-by: Guilhem Barthes <guilhem.barthes@owkin.com>
Signed-off-by: Guilhem Barthes <guilhem.barthes@owkin.com>
Signed-off-by: Guilhem Barthes <guilhem.barthes@owkin.com>
Signed-off-by: Guilhem Barthes <guilhem.barthes@owkin.com>
Signed-off-by: Guilhem Barthes <guilhem.barthes@owkin.com>
Signed-off-by: Guilhem Barthes <guilhem.barthes@owkin.com>
Signed-off-by: Guilhem Barthes <guilhem.barthes@owkin.com>
Signed-off-by: Guilhem Barthes <guilhem.barthes@owkin.com>
@guilhem-barthes guilhem-barthes force-pushed the fix/improve-dataset-loading branch from 067ceaf to 5ff3c00 Compare March 24, 2023 16:15
Signed-off-by: Guilhem Barthes <guilhem.barthes@owkin.com>
Signed-off-by: Guilhem Barthes <guilhem.barthes@owkin.com>
@guilhem-barthes
Copy link
Copy Markdown
Contributor Author

/e2e --tests sdk,frontend,substrafl --benchmarks mnist,camelyon

@Owlfred
Copy link
Copy Markdown

Owlfred commented Mar 24, 2023

End to end tests: ✔️ SUCCESS

Copy link
Copy Markdown
Member

@ThibaultFy ThibaultFy left a comment

Choose a reason for hiding this comment

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

LGTM

@guilhem-barthes guilhem-barthes merged commit 9381cb0 into main Mar 24, 2023
@guilhem-barthes guilhem-barthes deleted the fix/improve-dataset-loading branch March 24, 2023 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants