-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Addresses #4226 . Fixes problem when loading NormalizerTransformer from disk. #4321
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
4d7609e
Fixed problem when saving and loading normalizer transformers that wo…
antoniovs1029 89a54ef
Added test of normalizer loaded from disk for a multidimensional vector
antoniovs1029 7217cbf
Update test case
antoniovs1029 50baf55
Updated test
antoniovs1029 0406afd
Cleaning up
antoniovs1029 0699296
Added test for backward compatibility with normalizer transformer wit…
antoniovs1029 0982fb4
Added TODO comment to test
antoniovs1029 7f53c82
Changed the order of saving and loading ItemKind byte
antoniovs1029 d5639e7
Updating dependency on models repo to use one model in a backwardcomp…
antoniovs1029 da59714
Fixed test that checks backward compatibility of normalizer transformer
antoniovs1029 bc52016
Moved assert statement and removed redundant cast
antoniovs1029 5d61c22
Avoid loading column 'float0' which was actually not used by the tran…
antoniovs1029 662bb9d
Fixed indentation
antoniovs1029 0628770
Added test reproducing the original scenario
antoniovs1029 8975755
Changed visibility of classes made for test
antoniovs1029 58916b8
Minor fixes in the test case
antoniovs1029 87364c2
Changed "Fact" to "OnnxFact" on test from DnnImageFeaturizerTest.cs
antoniovs1029 2ac5b8e
Merge branch 'master' of https://github.com/dotnet/machinelearning in…
antoniovs1029 db96aaa
Merge remote-tracking branch 'upstream/master' into is04normalizer
antoniovs1029 740f267
Use WriteIntArray and ReadIntArray extension methods to make code mor…
antoniovs1029 499bf91
Merge remote-tracking branch 'upstream/master' into is04normalizer
antoniovs1029 6614c5f
Merge remote-tracking branch 'upstream/master' into is04normalizer
antoniovs1029 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Minor fixes in the test case
- Loading branch information
commit 58916b8dfc08a18fcee1a93f01fd80fd6dcb3076
There are no files selected for viewing
This file contains hidden or 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
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.
Uh oh!
There was an error while loading. Please reload this page.