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
Support for new keras layers #91
Conversation
* Changed `setParams` to allow greying out parameters * Added a flag to each parameter value to indicate whether to disable or not * Added 1D and 3D conv options to conv dropdown * Modified shape calculation to handle different dimensional data * Added import and export of Conv1D and Conv3D to keras support
* Added MaxPooling1D, MaxPooling3D, AveragePooling1D, AveragePooling3D, GlobalMaxPooling1D, GlobalAveragePooling1D * Added ZeroPadding1D, ZeroPadding3D
* Added ThresholdedReLU, selu, softplus, softsign and hard sigmoid
* GaussianNoise, GaussianDropout, AlphaDropout
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.
Looks very good overall. Great job!
@@ -185,7 +185,7 @@ class Canvas extends React.Component { | |||
layer.connection = { input: [], output: [] }; | |||
layer.params = {}; | |||
Object.keys(data[type].params).forEach(j => { | |||
layer.params[j] = data[type].params[j].value; | |||
layer.params[j] = [data[type].params[j].value, false]; |
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.
Comment mentioning that true maps to disabled?
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.
Its there in content.js line 287
@@ -185,7 +185,7 @@ class Canvas extends React.Component { | |||
layer.connection = { input: [], output: [] }; | |||
layer.params = {}; | |||
Object.keys(data[type].params).forEach(j => { | |||
layer.params[j] = data[type].params[j].value; | |||
layer.params[j] = [data[type].params[j].value, false]; | |||
}); | |||
// l.props = JSON.parse(JSON.stringify(data[type].props)); |
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.
Remove this
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.
its required I think
} | ||
}); | ||
if (type == 'Convolution' || type == 'Pooling' || type == 'Upsample' || type == 'LocallyConnected' || type == 'Eltwise'){ | ||
layer = this.adjustParameters(layer, 'layer_type', layer.params['layer_type'][0]); | ||
} | ||
// layer.props = JSON.parse(JSON.stringify(data[type].props)); |
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.
Can remove this
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.
you want to send all layers to adjustParameters ?
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.
Nevermind, some issue with the review interface
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.
Nevermind, some issue with the review interface
No description provided.