forked from tensorflow/java
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Metrics Phase 1 (#180) #9
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Initial checkin * Initial checkin and sync with master * Initial checkin and sync with master * JavaDoc cleanup * Javadoc fixes * Change LossInterface to LossMetric. Fix JavaDoc, modify one line code block to include braces. * Removed hashmap for variables, they are not needed as the variables only live within a single instance of a Metric. * reformat code * Add tests for assertBroadcastable * Change type to resultType * Added V data type for sampleWeights so that it is not forced to be the same type as the return or internal variables, * change 'type' to 'resultType' * clean up mean and fix assert assertBroadcastable * fix error message * Change sampleWeights to have its own generic type <S extends TNumber> * Add commment about invalid tests expecting IllegalArgumentExceptions * Add this exception instead of the more generic IllegalArgumentException when static shapes cannot boradcast. * change IllegalArgumentException to NotBroadcastableException. change hasValidNonscalarShape to canBroadcastNonscalarShapes change hasValidNonscalarShape to canBroadcastNonscalarShapes * reformat code * Fis=x Javadoc move the dynamic shapes and rank down to the dynamic section so they are created needlessly when static Fix if statement to check for unknown size and unknown dimensions * Fix Reduce to use boradcastWeights, renamed WeightBroadcastTest to AssertBroadcastableTest and added BroadcastWeightsTest * Added comment to count to indicate that it may be weighted. * Added SetsOps and fixed AssertBroadcastable to use SetsOps methods, * Fixed based on various PR comments. * Deleted, no longer needed after change to Variable handling in Metrics. * Nicer error messages for mode-forbidden ops (#169) * start fobbiden ops checks Signed-off-by: Ryan Nett <rnett@calpoly.edu> * fix style Signed-off-by: Ryan Nett <rnett@calpoly.edu> * move checks to builder method Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Initialization imprvements (#178) * No-op on initAdd in eager mode Signed-off-by: Ryan Nett <rnett@calpoly.edu> * runInit() method in session Signed-off-by: Ryan Nett <rnett@calpoly.edu> * add doInitialization() to Runner Signed-off-by: Ryan Nett <rnett@calpoly.edu> * fix javadoc Signed-off-by: Ryan Nett <rnett@calpoly.edu> * assume only graph or eager environments Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Remove doInit(), update javadocs Signed-off-by: Ryan Nett <rnett@calpoly.edu> * small fixes Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Clairify tensorOf lifetime requirements (#190) * Clairify tensorOf lifetime requirements Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Do codegen Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Remove extra generics from op generation (#193) * Successfully remove extra type params, but it broke javadoc generation Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Generate covariant types Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Do generation Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Update help text. Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Fixes Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Add Java 11 support - Initial Phase (#185) * Add profile for JDK11 and Automatic-Module-Name to jars * add maven.compiler.release=11 * Update manual ops for new codegen (#196) Signed-off-by: Ryan Nett <rnett@calpoly.edu> * Fix Losses to use CHANNELS_FIRST/LAST for CategoricalCrossentropy * Fix SetOps to properly convert sparse tensor to dense tensor using tf.sparse.sparseToDense with the output of tf.sparse.denseToDenseSetOperation * Initial checkin * Initial checkin and sync with master * Initial checkin and sync with master * JavaDoc cleanup * Javadoc fixes * Change LossInterface to LossMetric. Fix JavaDoc, modify one line code block to include braces. * Removed hashmap for variables, they are not needed as the variables only live within a single instance of a Metric. * reformat code * Add tests for assertBroadcastable * Change type to resultType * Added V data type for sampleWeights so that it is not forced to be the same type as the return or internal variables, * change 'type' to 'resultType' * clean up mean and fix assert assertBroadcastable * fix error message * Change sampleWeights to have its own generic type <S extends TNumber> * Add commment about invalid tests expecting IllegalArgumentExceptions * Add this exception instead of the more generic IllegalArgumentException when static shapes cannot boradcast. * change IllegalArgumentException to NotBroadcastableException. change hasValidNonscalarShape to canBroadcastNonscalarShapes change hasValidNonscalarShape to canBroadcastNonscalarShapes * reformat code * Fis=x Javadoc move the dynamic shapes and rank down to the dynamic section so they are created needlessly when static Fix if statement to check for unknown size and unknown dimensions * Fix Reduce to use boradcastWeights, renamed WeightBroadcastTest to AssertBroadcastableTest and added BroadcastWeightsTest * Added comment to count to indicate that it may be weighted. * Added SetsOps and fixed AssertBroadcastable to use SetsOps methods, * Fixed based on various PR comments. * Deleted, no longer needed after change to Variable handling in Metrics. * Fix Losses to use CHANNELS_FIRST/LAST for CategoricalCrossentropy * Fix SetOps to properly convert sparse tensor to dense tensor using tf.sparse.sparseToDense with the output of tf.sparse.denseToDenseSetOperation Co-authored-by: Ryan Nett <rnett@calpoly.edu>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Initial checkin
Initial checkin and sync with master
Initial checkin and sync with master
JavaDoc cleanup
Javadoc fixes
Change LossInterface to LossMetric.
Fix JavaDoc,
modify one line code block to include braces.
Removed hashmap for variables, they are not needed as the variables only live within a single instance of a Metric.
reformat code
Add tests for assertBroadcastable
Change type to resultType
Added V data type for sampleWeights so that it is not forced to be the same type as the return or internal variables,
change 'type' to 'resultType'
clean up mean and fix assert assertBroadcastable
fix error message
Change sampleWeights to have its own generic type
Add commment about invalid tests expecting IllegalArgumentExceptions
Add this exception instead of the more generic IllegalArgumentException when static shapes cannot boradcast.
change IllegalArgumentException to NotBroadcastableException.
change hasValidNonscalarShape to canBroadcastNonscalarShapes
change hasValidNonscalarShape to canBroadcastNonscalarShapes
reformat code
Fis=x Javadoc
move the dynamic shapes and rank down to the dynamic section so they are created needlessly when static
Fix if statement to check for unknown size and unknown dimensions
Fix Reduce to use boradcastWeights,
renamed WeightBroadcastTest to AssertBroadcastableTest and added BroadcastWeightsTest
Added comment to count to indicate that it may be weighted.
Added SetsOps and fixed AssertBroadcastable to use SetsOps methods,
Fixed based on various PR comments.
Deleted, no longer needed after change to Variable handling in Metrics.
Nicer error messages for mode-forbidden ops (Nicer error messages for mode-forbidden ops tensorflow/java#169)
start fobbiden ops checks
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Initialization imprvements (Initialization imprvements tensorflow/java#178)
No-op on initAdd in eager mode
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Clairify tensorOf lifetime requirements (Clairify tensorOf lifetime requirements tensorflow/java#190)
Clairify tensorOf lifetime requirements
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Remove extra generics from op generation (Remove extra generics from op generation tensorflow/java#193)
Successfully remove extra type params, but it broke javadoc generation
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Signed-off-by: Ryan Nett rnett@calpoly.edu
Add Java 11 support - Initial Phase (Add Java 11 support - Initial Phase tensorflow/java#185)
Add profile for JDK11 and Automatic-Module-Name to jars
add maven.compiler.release=11
Update manual ops for new codegen (Update manual ops for new codegen tensorflow/java#196)
Signed-off-by: Ryan Nett rnett@calpoly.edu
Fix Losses to use CHANNELS_FIRST/LAST for CategoricalCrossentropy
Fix SetOps to properly convert sparse tensor to dense tensor using tf.sparse.sparseToDense with the output of tf.sparse.denseToDenseSetOperation
Initial checkin
Initial checkin and sync with master
Initial checkin and sync with master
JavaDoc cleanup
Javadoc fixes
Change LossInterface to LossMetric.
Fix JavaDoc,
modify one line code block to include braces.
Removed hashmap for variables, they are not needed as the variables only live within a single instance of a Metric.
reformat code
Add tests for assertBroadcastable
Change type to resultType
Added V data type for sampleWeights so that it is not forced to be the same type as the return or internal variables,
change 'type' to 'resultType'
clean up mean and fix assert assertBroadcastable
fix error message
Change sampleWeights to have its own generic type
Add commment about invalid tests expecting IllegalArgumentExceptions
Add this exception instead of the more generic IllegalArgumentException when static shapes cannot boradcast.
change IllegalArgumentException to NotBroadcastableException.
change hasValidNonscalarShape to canBroadcastNonscalarShapes
change hasValidNonscalarShape to canBroadcastNonscalarShapes
reformat code
Fis=x Javadoc
move the dynamic shapes and rank down to the dynamic section so they are created needlessly when static
Fix if statement to check for unknown size and unknown dimensions
Fix Reduce to use boradcastWeights,
renamed WeightBroadcastTest to AssertBroadcastableTest and added BroadcastWeightsTest
Added comment to count to indicate that it may be weighted.
Added SetsOps and fixed AssertBroadcastable to use SetsOps methods,
Fixed based on various PR comments.
Deleted, no longer needed after change to Variable handling in Metrics.
Fix Losses to use CHANNELS_FIRST/LAST for CategoricalCrossentropy
Fix SetOps to properly convert sparse tensor to dense tensor using tf.sparse.sparseToDense with the output of tf.sparse.denseToDenseSetOperation
Co-authored-by: Ryan Nett rnett@calpoly.edu