NVIDIA-BioNeMo / NVIDIA-BioNeMo/nvMolKit
Enable clang-tidy builds again
Open
Nobody has claimed this yet.
Cleanup
- Dominant language
- Cuda
- Stars
- 276
- Forks
- 28
- Avg merge
- 2d 6h
- Merged PRs (30d)
- 26
Description
At least the performance sections. We used to have this, but it was causing more hassle than the code quality was worth, but we can add back in the perf/correctness things without a lot of noise from the other best practice settings
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the existing clang-tidy configuration and any prior setup in the repository, then identify how the project builds its C++/CUDA code. Re-enable only the performance and correctness checks requested, avoiding the broader best-practice settings, and verify that the relevant builds complete without excessive diagnostics.
Written by the indexing model from the issue text.
Assessment
- Domain
- build-system, tooling
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Needs clarification
- Newbie friendliness
- 45/100