提交 32f639d7 authored 作者: Frederic Bastien's avatar Frederic Bastien

modified the ConvOp hash calculation. If we don't consider the unroll_* option,…

modified the ConvOp hash calculation. If we don't consider the unroll_* option, we won't be able to correctly test the op as they will be merged together and we will test only the first version. This don't uncover error.
上级 c3210104
...@@ -17,7 +17,8 @@ class ConvOp(Op): ...@@ -17,7 +17,8 @@ class ConvOp(Op):
""" """
__attrnames = ['imshp', 'kshp', 'nkern', 'bsize', 'dx', 'dy', 'out_mode']
__attrnames = ['imshp', 'kshp', 'nkern', 'bsize', 'dx', 'dy', 'out_mode', 'unroll_batch', 'unroll_kern']#FRED: I added both unroll as we don't want ops to be merged if they have different value. Otherwise, the tests for the unroll don't work correctly.
"""These attributes uniquely identify the behaviour of this op for given inputs""" """These attributes uniquely identify the behaviour of this op for given inputs"""
def __init__(self, imshp, kshp, nkern, bsize, dx, dy, output_mode='valid', unroll_batch=0, unroll_kern=0): def __init__(self, imshp, kshp, nkern, bsize, dx, dy, output_mode='valid', unroll_batch=0, unroll_kern=0):
......
Markdown 格式
0%
您添加了 0 到此讨论。请谨慎行事。
请先完成此评论的编辑!
注册 或者 后发表评论