dotnet / dotnet/machinelearning-samples
Stop using version macros in sample projects
- 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