NetSpec: don't require lists to specify single-element repeated fields #2959

Merged
merged 1 commit into from Sep 3, 2015
Jump to file or symbol
Failed to load files and symbols.
+9 −4
Split
View
@@ -56,8 +56,14 @@ def to_proto(*tops):
def assign_proto(proto, name, val):
"""Assign a Python object to a protobuf message, based on the Python
type (in recursive fashion). Lists become repeated fields/messages, dicts
- become messages, and other types are assigned directly."""
-
+ become messages, and other types are assigned directly. For convenience,
+ repeated fields whose values are not lists are converted to single-element
+ lists; e.g., `my_repeated_int_field=3` is converted to
+ `my_repeated_int_field=[3]`."""
+
+ is_repeated_field = hasattr(getattr(proto, name), 'extend')
+ if is_repeated_field and not isinstance(val, list):
+ val = [val]
if isinstance(val, list):
if isinstance(val[0], dict):
for item in val:
@@ -43,8 +43,7 @@ def anon_lenet(batch_size):
def silent_net():
n = caffe.NetSpec()
- n.data, n.data2 = L.DummyData(shape=[dict(dim=[3]), dict(dim=[4, 2])],
- ntop=2)
+ n.data, n.data2 = L.DummyData(shape=dict(dim=3), ntop=2)
n.silence_data = L.Silence(n.data, ntop=0)
n.silence_data2 = L.Silence(n.data2, ntop=0)
return n.to_proto()