dotnet / dotnet/machinelearning-samples

Stop using version macros in sample projects

Aperta
#622 0 commenti 4 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
PowerShell
Stelle
4.7k
Fork
2.7k
Merge medio
2g 22h
PR unite (30g)
1

Descrizione

I am seeing [this pattern](https://github.com/dotnet/machinelearning-samples/blob/master/samples/csharp/getting-started/Ranking_Web/WebRanking/WebRanking.csproj#L9) in sample projects. I consider it an anti-pattern. You cannot just copy these samples and use them. We don't use this pattern elsewhere.

```csharp




```

The macros are defined in the root of the samples directory in [Directory.Build.props](https://github.com/dotnet/machinelearning-samples/blob/master/samples/Directory.Build.props).

I also found examples where it is [done wrong](https://github.com/dotnet/machinelearning-samples/blob/master/samples/csharp/getting-started/Regression_AutoML/TaxiFarePrediction/TaxiFarePrediction.csproj#L14):

```csharp

```

I made a [PR to add dependabot support for this project](https://github.com/dotnet/machinelearning-samples/pull/621). The dependabot model is a much better user experience than using these macros. The downside is that it will create a lot of PRs that need to be managed, but that's just a function of the number of samples provided.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia da samples/Directory.Build.props e dai file WebRanking.csproj e TaxiFarePrediction.csproj referenziati, quindi cerca MicrosoftMLVersion e MicrosoftMLPreviewVersion nei progetti di esempio. Il lavoro è concluso quando i progetti di esempio non dipendono più da macro di versione e usano riferimenti ai pacchetti che possono essere copiati e usati direttamente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
csharp
Ambito
build-system, machine-learning
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
42/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.