The Wayback Machine - https://web.archive.org/web/20201216164012/https://github.com/dotnet/machinelearning-samples/pull/617
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

DatabaseLoader sample loading an IDataView from SQL Server localdb #617

Merged
merged 12 commits into from Sep 2, 2019

Conversation

@CESARDELATORRE
Copy link
Contributor

@CESARDELATORRE CESARDELATORRE commented Aug 21, 2019

DatabaseLoader sample loading an IDataView from SQL Server localdb

Using public preview.

@CESARDELATORRE CESARDELATORRE merged commit bc306e5 into dotnet:master Sep 2, 2019
27 checks passed
27 checks passed
ML.NET Samples CI Build #20190902.2 succeeded
Details
ML.NET Samples CI (BikeSharingDemand) BikeSharingDemand succeeded
Details
ML.NET Samples CI (CreditCardFraudDetection) CreditCardFraudDetection succeeded
Details
ML.NET Samples CI (CreditCardFraudDetection2) CreditCardFraudDetection2 succeeded
Details
ML.NET Samples CI (CustomerSegmentation) CustomerSegmentation succeeded
Details
ML.NET Samples CI (DatabaseIntegration) DatabaseIntegration succeeded
Details
ML.NET Samples CI (GitHubLabeler) GitHubLabeler succeeded
Details
ML.NET Samples CI (HeartDiseaseDetection) HeartDiseaseDetection succeeded
Details
ML.NET Samples CI (IrisClustering) IrisClustering succeeded
Details
ML.NET Samples CI (IrisMultiClass) IrisMultiClass succeeded
Details
ML.NET Samples CI (Mnist) Mnist succeeded
Details
ML.NET Samples CI (MovieRecommenderE2E) MovieRecommenderE2E succeeded
Details
ML.NET Samples CI (MovieRecommenderMF) MovieRecommenderMF succeeded
Details
ML.NET Samples CI (ObjectDetectionConsoleApp) ObjectDetectionConsoleApp succeeded
Details
ML.NET Samples CI (ObjectDetectionE2EAPP) ObjectDetectionE2EAPP succeeded
Details
ML.NET Samples CI (PowerConsumptionAnomalyDetection) PowerConsumptionAnomalyDetection succeeded
Details
ML.NET Samples CI (SalesSpikeAndChangeDetection) SalesSpikeAndChangeDetection succeeded
Details
ML.NET Samples CI (SalesSpikeChangeDetectionE2E) SalesSpikeChangeDetectionE2E succeeded
Details
ML.NET Samples CI (ScalableMLModelOnWebAPIIntegrationPkg) ScalableMLModelOnWebAPIIntegrationPkg succeeded
Details
ML.NET Samples CI (SentimentAnalysis) SentimentAnalysis succeeded
Details
ML.NET Samples CI (SpamDetection) SpamDetection succeeded
Details
ML.NET Samples CI (TaxiFarePrediction) TaxiFarePrediction succeeded
Details
ML.NET Samples CI (TensorFlowFeaturizerEstimator) TensorFlowFeaturizerEstimator succeeded
Details
ML.NET Samples CI (TensorFlowScorer) TensorFlowScorer succeeded
Details
ML.NET Samples CI (eShopDashboardML) eShopDashboardML succeeded
Details
WIP Ready for review
Details
license/cla All CLA requirements met.
Details
Copy link

@divega divega left a comment

@CESARDELATORRE, I did a deferred review. The experience seems pretty good. And I agree with you that it could be even better with some sugar method that loads directly from the arguments of DatabaseSource.

I can also see other possible shortcuts of similar nature. For example, although DbProviderFactory is the all encompassing root concept if you need everything from an ADO.NET provider, a DbConnection can give you everything but DbConnectionStringBuilder, and it is a much more familiar abstraction for most users than the DbProviderFactory. So, unless you need to manipulate connection strings in a provider agnostic way (not commonly a useful thing to do), you could make the sugar Load method generic on the provider's DbConnection type.

All in all, I would would love to meet with you and the devs and walk trough the product code and API. It is likely that more things like this will come up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.