Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Update Bb line counting script #12350

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

codygunton
Copy link
Contributor

@codygunton codygunton commented Feb 27, 2025

I didn't think too hard about the weights, and I didn't go in and breakdown the subdirs of stdlib (tried to compensate in weighting). Still useful IMO

% python3 scripts/line_count.py
Excluding the following directories from the audit: ['acir_formal_proofs', 'bb', 'benchmark', 'boomerang_value_detection', 'common', 'env', 'examples', 'grumpkin_srs_gen', 'lmdblib', 'messaging', 'nodejs_module', 'plonk', 'smt_verification', 'vm', 'vm2', 'wasi', 'world_state']
Dirs with higher weight: []
    1655 text files.
    1644 unique files.
     156 files ignored.
Wrote build/per_file_report.csv
Total number of unweighted code lines to be audited: 147535
Total number of   weighted code lines to be audited: 161677

```
% python3 scripts/line_count.py
Excluding the following directories from the audit: ['acir_formal_proofs', 'bb', 'benchmark', 'boomerang_value_detection', 'common', 'env', 'examples', 'grumpkin_srs_gen', 'lmdblib', 'messaging', 'nodejs_module', 'plonk', 'smt_verification', 'vm', 'vm2', 'wasi', 'world_state']
Dirs with higher weight: []
    1655 text files.
    1644 unique files.
     156 files ignored.
Wrote build/per_file_report.csv
Total number of unweighted code lines to be audited: 147535
Total number of   weighted code lines to be audited: 161677
```
@codygunton codygunton force-pushed the cg/update-line-counter branch from 98b13a5 to deb715b Compare February 27, 2025 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant