dotnet/machinelearning

[DatabaseLoader] Create higher level convenient methods for DatabaseLoader

開放

#4,182 建立於 2019年9月6日

 (2 則留言) (0 個反應) (0 位負責人)C# (1,901 個分叉)batch import
P2enhancementgood first issue

倉庫指標

星標
 (9,125 顆星)
PR 合併指標
 (平均合併 6天 19小時) (30 天內合併 5 個 PR)

描述

As mentioned by Diego, these additions would help by simplifying the API usage for users even further and it should be pretty easy to implement for us: 👍

@divega commented: https://github.com/dotnet/machinelearning-samples/pull/617#pullrequestreview-284597248

@CESARDELATORRE, I did a deferred review. The experience seems pretty good.

1: And I agree with you that it could be even better with some sugar method that loads directly from the arguments of DatabaseSource.

2: 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 an 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.

貢獻者指南