Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Implement ModularDNN model #235
Implement ModularDNN model #235
Changes from all commits
966b9c9
ac7367e
790752c
63bdc1c
491fb17
7b75387
dff29ef
5367ddc
e3d550f
c389d5f
dbe9aed
eb2cf99
0a4d35f
f375170
88d3dbb
a953ccf
d2461e5
e505964
4a06b76
0b02a66
c247fa3
2aca8f9
ff50a6d
889eb30
846ac7d
3478830
a56dcb8
124588a
d4d8de1
6ccfacf
6282cf0
2edaf90
4312ea2
9b39da3
c32f3b3
3f85669
448929c
38f7b0d
de20c74
eabc54f
664ab73
65636fb
d6270ce
c2cd73e
0523dab
4625209
0ef4bc4
e64dd63
eeef53b
65a79cc
48aa4bd
8478ae8
92981a5
3181f0b
02bba38
3446703
c0ecec6
20464bc
c31b390
f41d285
e7a7b6d
ef8e463
3fcbe88
b9da8ab
7a79796
0d889ae
4c01b54
fc81052
b8229cf
dc0da16
c7c2240
d321709
e1f41a5
8828586
3590a80
01e9ebf
e4a7096
a5bab35
c56ec52
83e752e
4228fc6
19206e2
e501ac8
0284b57
b70459a
5e53560
bde5077
4242d5f
5d5fa86
a196914
4553338
cdbea96
d789202
cebf9af
fdb21c0
2849565
8e2099f
0c21bd6
3a18054
32e7725
5589d67
7e8cd8f
3e69ed8
73bc305
bae23b3
1966be0
c6de03a
c52b9ff
9b79a59
0c4b814
1ae3103
c5e0918
de514f2
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: better for this to be a tuple since it must be len 2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. However, tuple's don't seem to be supported by default in yaml.
You could add a validator like this:
Or one that converts the list to a tuple and ensures that it has length two. Haven't made either of these changes because there is a tonne of validation that we haven't really bothered to do in these configs, I don't think it's very high priority as most of them should just raise errors later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh yeah, that's fine then.