Clippy linting is very slow
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- rust
- Domaine
- compilers, performance
Piste de recherche
Commencez par comparer les exécutions CI liées, en vous concentrant sur build.rs du crate GPU et sur l'appel à SpirvBuilder::clippy(). Inspectez les dépendances SPIR-V répertoriées dans crates/farmer/ab-proof-of-space-gpu/Cargo.toml. Le travail est terminé lorsque vous avez identifié pourquoi Clippy ajoute environ huit minutes à ab-farmer CI et réduit cette régression avec des éléments issus d'une nouvelle exécution.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
I switched to using clippy for linting GPU code in CI and I think it increased CI time by ~8 minutes.
I don't think it should take 8 minutes to run cargo clippy though.
Here is CI run where build.rs for GPU crate was skipping all the work under Clippy: https://github.com/nazar-pc/abundance/actions/runs/21787689837/job/62861720690#step:11:1022
Here is CI run where SpirvBuilder::clippy() is called: https://github.com/nazar-pc/abundance/actions/runs/21905750349/job/63245490660#step:11:1023
Clippy time for ab-farmer crate that includes ab-proof-of-space-gpu as a dependency increased from ~2 minutes to over 10 minutes.
Dependencies for SPIR-V are fairly minimal too and should not take nearly as much time to compile and lint: https://github.com/nazar-pc/abundance/blob/c55eae7482783aea52f0f1e9510e71378c9f10b8/crates/farmer/ab-proof-of-space-gpu/Cargo.toml#L19-L23
I do not know what could it possibly be doing for such a long time, but I do not like it.
- Langage dominant
- Rust
- Étoiles
- 3.4k
- Forks
- 126
- Merge moyen
- 1 h 44 min
- PR mergées (30 j)
- 2
Préparer son environnement
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Rust-GPU/rust-gpu
-
enhancement
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
-
Adhere to Rust floating-point semanticsPeut-être pris @Firestar99 l’a pris il y a 14 jours. Ouverteenhancement
Rust-GPU/rust-gpu#643 · 3 commentaires · 1 réaction · 1 personne assignée ·
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
-
async functions cause ICEOuvertebug
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
Toutes les issues de Rust-GPU/rust-gpu
Issues similaires
-
`sysknife history --help` says --since takes ISO-8601, and the parser refuses offsets and bare datesOuvertebug easy good first issue help wanted
Difficulté 1/5 1-3 heures Accessibilité débutants 94/100
lacs-project/sysknife#519 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
-
area:breg bug criticality:p3 triage:needs-implementation
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
registrystack/registry-stack#1699 ·
Les mainteneurs répondent en général sous 1 jour
-
documentation
Difficulté 1/5 1-3 heures Accessibilité débutants 84/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
lbjlaq/Antigravity-Manager#3539 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour