claimed-framework/mlx

Add server stand up test as part of MLX CI

Aperta

#213 aperta il 15 set 2021

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (1 assegnatario)Python (54 fork)github user discovery
APIhelp wantedsize/Stest

Metriche repository

Star
 (218 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Hi @ckadner sorry I didn't have time to recompile the server for every PR. Right now our CI is only catching build errors but we don't have any test to catch runtime errors. Maybe we can add a simple test just to start the server to make sure nothing is crashing the server startup? This way we don't accidentally merged critical bugs that crashes our server startup.

Originally posted by @Tomcli in https://github.com/machine-learning-exchange/mlx/issues/205#issuecomment-920338732

Guida contributor

Direzione di ricerca
Identifica lo script o comando di avvio del server, crea un test minimo che lo esegua e verifichi l'avvio riuscito, quindi integralo nella CI.
Tech stack
python
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75