Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

WIP: Loading#1

Merged
justusschock merged 40 commits intomasterfrom
loading
Dec 4, 2019
Merged

WIP: Loading#1
justusschock merged 40 commits intomasterfrom
loading

Conversation

@justusschock
Copy link
Member

@justusschock justusschock commented Nov 26, 2019

I've started adding the loader functionality to this repo. I also left TODOs where anything is still missing.

In particular these are:

  • Docstrings for the containers
  • Docstrings for the ID Managers
  • Docstrings for the Splitter Class
  • More test cases for the numpy_collate
  • More tests for the subset dataset
  • Tests for the debug mode
  • Tests for the actual data loader (the things we changed)

Should we nevertheless merge it as is or wait until we have done all this @mibaumgartner ?

@justusschock justusschock merged commit 5880178 into master Dec 4, 2019
@justusschock justusschock deleted the loading branch December 4, 2019 08:28
p-wein pushed a commit to p-wein/rising that referenced this pull request Oct 15, 2020
Yuxiang1990 pushed a commit to Yuxiang1990/rising that referenced this pull request Nov 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants