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

TransferFunction: Add support for computing derivatives using base function output #2538

Merged
merged 8 commits into from
Nov 14, 2022

Conversation

jvesely
Copy link
Collaborator

@jvesely jvesely commented Nov 13, 2022

The results of SoftMax can be provided from outside, or calculated locally. Adjust the compiled variant, and the test results to match.
Add compiled implementation of Logistic derivative using outputs.
Implement ReLU derivative using outputs.

Add tests for variant computing derivatives using outputs.
Cleanup test naming and parameter passing.

Signed-off-by: Jan Vesely [email protected]

The results can be provided from outside, or calculated locally.
Adjust the compiled variant, and the test results to match.

Signed-off-by: Jan Vesely <[email protected]>
@jvesely jvesely added the compiler Runtime Compiler label Nov 13, 2022
@github-actions
Copy link

This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):

No differences!

...

See CI logs for the full diff.

…parameters

Don't use nested 'params'

Signed-off-by: Jan Vesely <[email protected]>
…tive using base output

Fix assertion in Python SoftmaxDerivative.
Add tests.

Signed-off-by: Jan Vesely <[email protected]>
…ase output

Both Python and compiled.
Add tests.

Signed-off-by: Jan Vesely <[email protected]>
@jvesely jvesely changed the title functions/SoftMax: Use "ALL" results to calculate derivative directly TransferFunction: Add support for computing derivatives using base function output Nov 14, 2022
@github-actions
Copy link

This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):

No differences!

...

See CI logs for the full diff.

…ative using base output

Add tests.

Signed-off-by: Jan Vesely <[email protected]>
@github-actions
Copy link

This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):

No differences!

...

See CI logs for the full diff.

@github-actions
Copy link

This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):

No differences!

...

See CI logs for the full diff.

@jvesely jvesely merged commit bc82bb1 into PrincetonUniversity:devel Nov 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler Runtime Compiler
Projects
Development

Successfully merging this pull request may close these issues.

1 participant