-
Notifications
You must be signed in to change notification settings - Fork 24
opt.inputDims
is a required argument but is not enforced
#14
Comments
Sorry about that. Can you try the examples, they should all run without I don’t think you can default the inputDims, but we can definitely throw a On Tue, Feb 9, 2016 at 3:27 PM, Victor Zhong notifications@github.com
|
In this line
https://github.com/twitter/torch-dataset/blob/master/lua/Batch.lua#L19
applyInputDims
is called withopt.inputDims
which by default is not set. HoweverapplyInputDims
expects this argument and crashes (eg. assert fails) if this is not set.I think there should be either
opt.inputDims
isopt.InputDims
.Otherwise the
README
example under Multiple Indices fail whensampledBatcher
is called:The text was updated successfully, but these errors were encountered: