{"push_id":2836733081,"size":15,"distinct_size":15,"ref":"refs/heads/master","head":"5ef7a082e2718874477d2f6e0119512ca0af9820","before":"2f4e50dc576552eef5ad8a7155080386c2931e53","commits":[{"sha":"105975b12cfb4f9e5cb6e8209fa7a6a5c31c3eee","author":{"name":"Adam Sitnik","email":"ae604ab1ff0e175f3db7a7921d207eb703f56422@gmail.com"},"message":"Few benchmark fixes (#703)\n\n* update to latest BenchmarkDotNet, x3 perf improvement\r\n\r\n* pass the arguments to BenchmarkSwitcher to get advantage of more BDN features\r\n\r\n* we must execute the IEnumerable to measure the actual execution time\r\n\r\n* run not 3, but up to 20 iterations\r\n\r\n* add README.md for the benchmarks\r\n\r\n* it's not mandatory to run against local dotnet cli, simplify the docs\r\n\r\n* move the BenchmarkDotNet version to Dependencies.props","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/105975b12cfb4f9e5cb6e8209fa7a6a5c31c3eee"},{"sha":"841ba785c58eb84fa0ce07de324f25d6e3dcbf94","author":{"name":"Eric Erhardt","email":"0477d1fb317f1f17936bbceb9830bc0c8ae0248f@microsoft.com"},"message":" Replace all ML.Transforms SubComponent usages with IComponentFactory. (#700)\n\n* Replace all ML.Transforms SubComponent usages with IComponentFactory.\r\n\r\nWorking towards #585\r\n\r\n* PR feedback","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/841ba785c58eb84fa0ce07de324f25d6e3dcbf94"},{"sha":"4fd8a9cb08b437397cbec4ec1d49456474309337","author":{"name":"Brian Lui","email":"ac81c7458b5cad70d975f4b25e95fbd0a697ddb8@microsoft.com"},"message":"Add new benchmarks to test\\Microsoft.ML.Benchmarks (#722)","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/4fd8a9cb08b437397cbec4ec1d49456474309337"},{"sha":"60ae981223e83b174ecaaf528bd51814a6b0835c","author":{"name":"Pete Luferenko","email":"df03d6710860d6cee9eded85c0befdfdd1bacb6d@users.noreply.github.com"},"message":"SDCA trainers become Estimators (#716)\n\nConverted three SDCA trainers to estimators","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/60ae981223e83b174ecaaf528bd51814a6b0835c"},{"sha":"fe5c0e2e329e48b83047b23c8c14100c6583450e","author":{"name":"Zeeshan Siddiqui","email":"0a597cd9391944a27bd610de4e4a785a9e5bf965@microsoft.com"},"message":"Add sigmoid to lightgbm parameter list and change default to 1. (#667)\n\n* Add sigmoid to lightgbm parameter list.","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/fe5c0e2e329e48b83047b23c8c14100c6583450e"},{"sha":"fe71bb82b8ef70e20f98d6fa77f246daba6f7a51","author":{"name":"Pete Luferenko","email":"df03d6710860d6cee9eded85c0befdfdd1bacb6d@users.noreply.github.com"},"message":"Turn TextLoader into a data reader (#723)\n\nTextLoader is now an `IDataReader<IMultiStreamSource>`","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/fe71bb82b8ef70e20f98d6fa77f246daba6f7a51"},{"sha":"4ba15b8387d1ed45bf7a49cbee67b445cb3a5fd2","author":{"name":"Ivan Matantsev","email":"6904773a27be812a718c6238a1208d1c8268428d@gmail.com"},"message":"Implementing copy column estimator (#706)","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/4ba15b8387d1ed45bf7a49cbee67b445cb3a5fd2"},{"sha":"5e1cd571a2ae4bee782bf16d7d9fc43c9d54e792","author":{"name":"Zeeshan Siddiqui","email":"0a597cd9391944a27bd610de4e4a785a9e5bf965@microsoft.com"},"message":"IDV test for DvTypes (#750)\n\n* Add IDV test for backward compatiblity with DvTypes.","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/5e1cd571a2ae4bee782bf16d7d9fc43c9d54e792"},{"sha":"d9d27ac05b63f3bbe420714bb4620a424b17d78e","author":{"name":"Eric Erhardt","email":"0477d1fb317f1f17936bbceb9830bc0c8ae0248f@microsoft.com"},"message":"Convert ML.Sweeper usages of SubComponent to IComponentFactory. (#734)","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/d9d27ac05b63f3bbe420714bb4620a424b17d78e"},{"sha":"98ac03b7e3210fab970b6983aee7f1fd4e021b7d","author":{"name":"Gani Nazirov","email":"df9efbb13c65a260b4d21fc748e1d11584eaffa7@hotmail.com"},"message":"HashTransform to handle Floats and Doubles (#684)\n\n* Added overrides for public boo Equals(Object o) & public int GetHashCode() for VectorType, KeyType and ImageType\r\n\r\n* fix build, remove breaking xml\r\n\r\n* fix comments\r\n\r\n* fix comments\r\n\r\n* fix comments\r\n\r\n* fix comments\r\n\r\n* Add unit test\r\n\r\n* fix test\r\n\r\n* Added unit test\r\n\r\n* remove check\r\n\r\n* Add HashTransform support for R4, R8\r\n\r\n* handle negative 0\r\n\r\n* handle negative 0\r\n\r\n* fix comments\r\n\r\n* fix more comments\r\n\r\n* fix comments\r\n\r\n* fix out of bounds exception also make if condition more readable\r\n\r\n* better unit test\r\n\r\n* improve on delegate\r\n\r\n* fix hasher\r\n\r\n* better name for dstLength","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/98ac03b7e3210fab970b6983aee7f1fd4e021b7d"},{"sha":"5a6fdedcc235deb02da72526097e5d0b9955ed2c","author":{"name":"Santiago Fernandez Madero","email":"998754c17f6827ebaffc18eaab073adbb353b587@microsoft.com"},"message":"Move macOS CI legs to VSTS (#727)\n\n* Move macOS CI builds to VSTS:public-CI\r\n\r\n* Update README badges","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/5a6fdedcc235deb02da72526097e5d0b9955ed2c"},{"sha":"8bf2610c3e27b9aeaede0ff26a02e18b5131e1ee","author":{"name":"Eric Erhardt","email":"0477d1fb317f1f17936bbceb9830bc0c8ae0248f@microsoft.com"},"message":"Convert ML.Data usages of SubComponent to IComponentFactory. (#729)\n\n* Convert ML.Data usages of SubComponent to IComponentFactory.\r\n\r\n* Update baselines now that AveragedPerceptron is no longer the default trainer.\r\n\r\n* PR feedback\r\n\r\n* Update baseline to fix failing test","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/8bf2610c3e27b9aeaede0ff26a02e18b5131e1ee"},{"sha":"4c3759a27f381efb51860aeca4c192a0a04b52a6","author":{"name":"Brian Lui","email":"ac81c7458b5cad70d975f4b25e95fbd0a697ddb8@microsoft.com"},"message":"Port all relevant AVX hardware intrinsics C# APIs from SIMD native algorithms (#691)\n\n* Implemented AVX intrinsics\r\n\r\n* Implemented performance tests for AVX intrinsics, with some fixes to the intrinsics\r\nNote: Building perf tests succeed, but running perf tests for AVX intrinsics ends without results.\r\n\r\n* Changes to perf tests in response to feedback\r\n\r\n* Fixes across multiple files to make unit tests and perf tests work for all used AVX intrinsics\r\nNote: Except matrix operations\r\n\r\n* Implemented new AVX intrinsics that do not involve matrix operations, passing basic unit tests\r\n\r\n* Implemented perf tests for AVX via CpuMathUtils class\r\n\r\n* Implemented switching logic for Vector128/256Alignment between SSE and AVX support\r\n\r\n* Changed perf tests to reveal SSE and AVX intrinsics perf separately\r\n\r\n* Fixed access modifiers of private fields\r\n\r\n* Implemented all unit tests for AVX intrinsics that do not involve matrix operations with longer input arrays\r\n\r\n* Implemented unit tests for AVX intrinsics\r\n\r\n* Fixed errors on the RffTransform.CfltAlign const-expression requirement\r\n\r\n* Fixed Debug errors by making RffTransform.CfltAlign read-only\r\n\r\n* Fixed errors by making CfltAlign static (and read-only)\r\n\r\n* Developed two unit tests for netcoreapp and netstandard to deal with different alignments separately, with some style changes to readonly variables\r\n\r\n* Kept only the most recent unit tests which are sufficient for both netcoreapp and netstandard\r\n\r\n* Respond to PR feedback: Style changes\r\n\r\n* Respond to PR feedback: More style changes\r\n\r\n* Implemented class inheritance in perf tests to reduce overlapping code\r\n\r\n* Respond to PR feedback: Changed Sse/AvxIntrinsics from public to interal, adding InternalsVisibleTo attributes\r\n\r\n* Respond to PR feedback: Used env vars to determine whether to use AVX/SSE\r\n\r\n* Respond to PR feedback: Included 0 into consideration for parsing env vars\r\n\r\n* Respond to PR feedback: env vars, InternalsVisibleTo, and abstract\r\n\r\n* Respond to PR feedback: Added new comparer class specifically for MatMul\r\n\r\n* Respond to PR feedback: Changes to intrinsics\r\n\r\n* Respond to PR comment: Makes alignment checking consistent in external and internal calls\r\n\r\n* Respond to PR feedback: Refactored Sse/AvxIntrinsics helper functions\r\n\r\n* Made two Sse/AvxIntrinsics helper functions about AlignedArray inline in hopes of improving perf\r\n\r\n* Respond to PR feedback: styles for Vector256Alignment and Avx.GetLowerHalf","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/4c3759a27f381efb51860aeca4c192a0a04b52a6"},{"sha":"9258be22e3338dec18075c6e66048b1861533ec8","author":{"name":"Eric Erhardt","email":"0477d1fb317f1f17936bbceb9830bc0c8ae0248f@microsoft.com"},"message":"Update buildtools to 3.0.0-preview1-03129-01 (#767)\n\n* Update buildtools to 3.0.0-preview1-03129-01\r\n\r\nThis new version has a fix for the Copyright header.\r\n\r\nFix #766\r\n\r\n* PR feedback","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/9258be22e3338dec18075c6e66048b1861533ec8"},{"sha":"5ef7a082e2718874477d2f6e0119512ca0af9820","author":{"name":"Abhishek Goswami","email":"937f61b659ed026b140b37d8f471547230ecac71@gmail.com"},"message":"TensorFlowMapper transform for scoring Tensorflow models in ML.NET (#704)\n\n* creating dummy file to test permissions. will remove\r\n\r\n* test\r\n\r\n* TensorFlow scoring, from Zeeshan A.'s branch, with some additional changes.\r\n\r\n* creating dummy file to test permissions. will remove\r\n\r\n* test\r\n\r\n* TensorFlow scoring, from Zeeshan A.'s branch, with some additional changes.\r\n\r\n* taking care of review comments; build fixes\r\n\r\n* simple change intended to trigger fresh builds to repro OSX-Release build failure\r\n\r\n* Prevent input tensors from being GC'ed before TF_SessionRun is called\r\n\r\n* Remove Tensorflow models from tests data\r\n\r\nInstead bring these from a nuget package created in another repo.\r\n\r\nFor now this is https://github.com/ericstj/machinelearning-testdata-temp\r\n\r\nSoon it will be https://github.com/dotnet/machinelearning-testdata\r\n\r\n* Add entry point\r\n\r\n* Fix manifest and generated C# API.\r\n\r\n* Create a redist package for tensorflow\r\n\r\nCreate a nuget package that redistributes the TensorFlow C-API.\r\n\r\nThis is needed because TensorFlow doesn't ship an official NuGet package.\r\n\r\nThis is a straight up repack of the bits published on tensorflow.org.  I made sure to apply the TensorFlow license to this package and not sign it with our authenticate certificates.\r\n\r\n* Make TF redist project a normal MSBuild project\r\n\r\nRemove the use of the SDK targets, and define our own build and clean.\r\n\r\n* Add TF License file to package\r\n\r\n* Don't use fullpaths when un-tar'ing\r\n\r\nTar on windows was failing when msbuild passed it a full path.  Workaround by using relative\r\npaths and running where we extract to.\r\n\r\n* Add some logging to TF redist project\r\n\r\n* Fix casing of TF redist proj\r\n\r\n* Change tests to use redistributed TensorFlow\r\n\r\nAlso modify TF binding code to use `tensorflow` in its DLLImports.\r\n\r\nThe runtime will still add the approriate prefix/extension on linux/mac.\r\n\r\n* Fix mac / linux tensorflow redist\r\n\r\nI was missing the libtensorflow_framework dependency which caused mac and linux tests to fail.\r\n\r\nAfter fixing that, mac still failed due to inability to load libtensorflow_framework.so.\r\n\r\nWe have to rename these to .dylib to satisfy the CORECLR dllimport convention which broke the internal rpath in libtensorflow which pointed at @rpath/libtensorflow_framework.so.  Fix this by rewriting the renamed libtensorflow.dylib on mac.\r\n\r\nSince this operation can only be done on mac, I had to change the build of the redist project to only build the bits appropriate for the building platform.  To make this work correctly in the official build I had to make sure these platform specific builds happen when the native build happens.\r\n\r\n* Only include LICENSE if it exists\r\n\r\nLICENSE is pulled from the Windows package, so it isn't available on mac or linux.\r\n\r\n* unit test to verify TF transform works with ML.NET image transforms\r\n\r\n* Factor TensorflowTransform into its own assembly/package\r\n\r\n* Update Tensorflow to TensorFlow\r\n\r\n* Fix manifest and C# API. Also, add entry point unit test.\r\n\r\n* update test case for image transforms; still skipping test for now till we figure out why false positive passes\r\n\r\n* Use IDataView instead of var in unit test.\r\n\r\n* fix have from Tensorflow to TensorFlow\r\n\r\n* Fix unit test to use new TextLoader APIs.\r\n\r\n* Add unit test using the pipeline API.\r\n\r\n* Validate input dimensions.\r\n\r\n* Added XML doc for TensorflowTransform.\r\n\r\n* Remove extra dimension for batch size in output.\r\n\r\n* Fix input/output validation.\r\n\r\n* Introduced BatchSize as constant to replace 1 everywhere.\r\n\r\n* enabling unit test of TensorFlowTransform working with ML.NET Image* Transforms\r\n\r\n* Extended XML docs with detailed information.\r\n\r\n* Ensure we include actual TF license in package\r\n\r\nThe TF zip/tarballs were missing the actual TF license.  Download this and include it in the package.  Rename the file from the zip/tarballs as THIRD_PARTY_NOTICES.txt as that represents its content.\r\n\r\n* Change input validation to validate all dimensions in case of multi dimensional input.\r\n\r\n* Corrected typos in the documentation.\r\n\r\n* Fix LICENSE inclusion in package.\r\n\r\nWe didn't define ExtractDirectory on the item and instead had a full path as identity.\r\n\r\nThis worked on linux/osx since it prepended a \"\"/ to the path, which was tolerated by the file system (an extra leading slash).\r\n\r\nOn Windows this doesn't work or course.\r\n\r\nFix by appending to the item that doesn't assume files came from an archive.\r\n\r\n* Add symbols package and fix package reference to redist\r\n\r\n* Address pull request comments.\r\n\r\n* Give more details in input dimension mismatch error message.\r\n\r\n* Added a test for LearningPipelineAPI and updated the doc.xml","distinct":true,"url":"https://api.github.com/repos/shauheen/machinelearning/commits/5ef7a082e2718874477d2f6e0119512ca0af9820"}]}
{"actor":{"display_login":"shauheen"}}