Skip to content
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

Make the first example for the Sequential model pass #4

Closed
cesarsouza opened this issue Aug 13, 2017 · 3 comments
Closed

Make the first example for the Sequential model pass #4

cesarsouza opened this issue Aug 13, 2017 · 3 comments

Comments

@cesarsouza
Copy link
Owner

cesarsouza commented Aug 13, 2017

Make the first example for a Sequential model given in https://keras.io/models/sequential/ pass as a unit test in the framework. The example is:

 model = Sequential()
 model.add(Dense(32, input_shape=(500,)))
 model.add(Dense(10, activation='softmax'))
 model.compile(optimizer='rmsprop',
       loss='categorical_crossentropy',
       metrics=['accuracy'])

and the unit test is located at https://github.com/cesarsouza/keras-sharp/blob/master/Tests/SequentialTest.cs#L28

cesarsouza added a commit that referenced this issue Aug 13, 2017
Updates GH-6: Implement the Flatten layer
Updates GH-5: Implement the Conv2D layer
Updates GH-4: Make the first example for the Sequential model pass
Updates GH-1: Contributing to Keras-Sharp
@cesarsouza
Copy link
Owner Author

Almost there. The next step is to implement a C# equivelent of the _collect_metrics function and put it in Model.cs, line 577:

private List<List<IMetric>> _collect_metrics(IMetric metrics, List<string> output_names)

@cesarsouza
Copy link
Owner Author

Done, the next goal now would be to implement a C# equivalent of the _masked_objective and put it in Model.cs, line 610:

private IMetric _masked_objective(IMetric metric_fn)

cesarsouza added a commit that referenced this issue Aug 22, 2017
…ceptions anymore. It is now necessary to compare the complete state of the Sequential model against Python do determine properties that have not been set correctly.

 - Updates GH-4: Make the first example for the Sequential model pass
@cesarsouza
Copy link
Owner Author

cesarsouza commented Aug 22, 2017

The execution path for the sequential model does not throw obvious exceptions anymore. It is now necessary to compare the complete state of the Sequential model against Python to determine properties that have not been set correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant