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.

Sign up for GitHub

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

Jump to bottom

Improve docs of sigmoid_cross_entropy #3562

Merged
merged 2 commits into from Oct 19, 2017
Merged

Improve docs of sigmoid_cross_entropy #3562

merged 2 commits into from Oct 19, 2017

Conversation

naoto0804
Copy link
Contributor

This is PR for improving docs of functions/links. Related issue: #2182

Modified functions/links:

  • chainer/functions/loss/sigmoid_cross_entropy

@naoto0804
Copy link
Contributor Author

screen_shot

@naoto0804
Copy link
Contributor Author

Could you review this? (In travis CI, it seems that one test not relavant to this PR is failing.)

@naoto0804 naoto0804 changed the title Improve docs of sigmoid cross entropy Improve docs of sigmoid_cross_entropy Oct 14, 2017
@niboshi niboshi added cat:document Documentation such as function documentations, comments and tutorials. to-be-backported Pull request that should be backported. st:test-and-merge State indicating that pull request is approved by a reviewer and can be merged after CI passes. labels Oct 18, 2017
@niboshi niboshi self-assigned this Oct 18, 2017
@niboshi
Copy link
Member

niboshi commented Oct 18, 2017

jenkins, test this please

@niboshi
Copy link
Member

niboshi commented Oct 19, 2017

LGTM!

@niboshi niboshi merged commit 90aa116 into chainer:master Oct 19, 2017
niboshi added a commit to niboshi/chainer that referenced this pull request Oct 19, 2017
@niboshi
Merge pull request chainer#3562 from naoto0804/improve-docs-of-sigmoi…
ee65ec3 
…d_cross_entropy

Improve docs of sigmoid_cross_entropy
@niboshi niboshi mentioned this pull request Oct 19, 2017
Merged
@niboshi niboshi added this to the v4.0.0b1 milestone Oct 19, 2017
@shiba24 shiba24 mentioned this pull request Nov 13, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@niboshi niboshi

Labels
cat:document Documentation such as function documentations, comments and tutorials. st:test-and-merge State indicating that pull request is approved by a reviewer and can be merged after CI passes. to-be-backported Pull request that should be backported.
Projects
None yet
Milestone
v4.0.0b1
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@naoto0804 @niboshi