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

For the dataset.py/init_callable, you shoul also catch Attribute error #79

Closed
nkundiushuti opened this issue Jun 3, 2015 · 1 comment

Comments

@nkundiushuti
Copy link

I have implemented the Largedataset class as you recommended, and the dataset method yields this error:
AttributeError: LargeDataset instance has no attribute 'len'
In my opinion, you should also catch this error in dataset.py
try:
self.iteration_size = len(samples)
except TypeError: # has no len
If you want, I can share the LargeDataset class with you, but it's a bit particular to my problem. I have added the len attribute to return the number of batches in one dataset file which is loaded from hard drive.

@lmjohns3
Copy link
Owner

lmjohns3 commented Jun 3, 2015

This is a good idea! In the meantime, I have moved the Dataset class to another package -- https://github.com/lmjohns3/downhill. It looks like there's not a quick way to move an issue to another repository, so I'm going to close this one here and open it there: lmjohns3/downhill#1

@lmjohns3 lmjohns3 closed this as completed Jun 3, 2015
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

No branches or pull requests

2 participants