Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

DL4J Docs Update 2.15.2019 #171

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

Closed
ejunprung wants to merge 1 commit into master from edstuff
Closed

DL4J Docs Update 2.15.2019 #171

ejunprung wants to merge 1 commit into master from edstuff

Conversation

Copy link
Contributor

@ejunprung ejunprung commented Feb 16, 2019

No description provided.

Copy link
Contributor Author

@crockpotveggies Check it out. Autogen seems to work perfectly for me now.

Copy link
Contributor

@crockpotveggies crockpotveggies left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deeplearning4j-nn-convolutional is broken and is not generating java classes properly. Please fix.

Copy link
Contributor

@crockpotveggies crockpotveggies left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deeplearning4j-nn-convolutional is broken and is not generating java classes properly. Please fix.

crop left depth, crop right depth, crop left height, crop right height, crop left width,
crop right width

{{autogenerated}}
Copy link
Contributor

@crockpotveggies crockpotveggies Feb 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Problem starts here.

<span style="float:right;"> [[source]](https://github.com/deeplearning4j/deeplearning4j/tree/master/deeplearning4j/deeplearning4j-nn/src/main/java/org/deeplearning4j/earlystopping/termination/ScoreImprovementEpochTerminationCondition.java) </span>

Terminate training if best model score does not improve for N epochs
{{autogenerated}}
Copy link
Contributor

@crockpotveggies crockpotveggies Feb 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same problem exists here.

however SimpleRnn is very fast to compute, and hence may be considered where the length of the temporal dependencies
in the dataset are only a few steps long.

{{autogenerated}}
Copy link
Contributor

@crockpotveggies crockpotveggies Feb 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same problem exists here.

Gets the current mask array from the provided input
- return The mask or null, if no input was provided

{{autogenerated}}
Copy link
Contributor

@crockpotveggies crockpotveggies Feb 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same problem exists here.

Copy link
Contributor Author

ejunprung commented Feb 16, 2019
edited
Loading

@crockpotveggies I'm confused. So {{autogenerated}} should automatically insert the Java classes, right?

If so, it might be an issue with the auto gen script. Does it happen to be working on your computer? It seemed to run fine on my Macbook but I'm not sure of the expected behavior.

FWIW all these files were already stashed in doc_sources. I only fixed a broken link on the evaluation page.

Copy link
Contributor

One possible explanation is that the java classes have moved locations. Not sure without looking at it further, can you copy/paste the output from the gen docs script and send it to me?

Copy link
Contributor Author

Copy link
Contributor

crockpotveggies commented Feb 16, 2019 via email

Yea there's an error in there I'm guessing a package changed. Best, Justin
...
On Feb 15, 2019, 11:12 PM -0800, ejunprung ***@***.***>, wrote: https://gist.github.com/ejunprung/2d74016bc1332bf6e61275a66bff3e6f — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

@sshepel sshepel deleted the edstuff branch July 29, 2019 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@crockpotveggies crockpotveggies crockpotveggies requested changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /