Skip to content

More comment stats #28

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.

Sign up for GitHub

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

Jump to bottom
Merged
patcon merged 34 commits into from
Apr 6, 2025
Merged

More comment stats #28

patcon merged 34 commits into from
Apr 6, 2025

Conversation

@patcon
Copy link
Member

@patcon patcon commented Mar 18, 2025 •
edited
Loading

To Dos

  • finish enumerating todos
  • add group-aware-consensus calculations
    • add unit tests
    • calculate for all statements
  • add importance_metric()
  • add priority_metric()
    • add unit tests
    • calculate for all statements
  • add extremity function and calculation
  • add importance_metric() function
    • add unit tests
    • calculate for all statements
  • convert group-specific metrics into multi-index (group+statements) dataframe
  • convert statement-specific metrics into (statements) dataframe

patcon added 15 commits March 21, 2025 20:59
@patcon
Add group-aware consensus calculation.
0cf582f
@patcon
Add and document importance_metric() and priority_metric() for commen…
7acc577 
…t priorities/routing.
@patcon
Add unit tests for priority_metric().
ac10052
@patcon
Add bugfix and test case for arrays in priority_metric(). Add docs.
87ed52f
@patcon
Allow pseudo-count to be a numpy array.
1e8caa6
@patcon
Added unit tests for importance_metric().
ef8bf96
@patcon
Add arg for pseudo-count into priority_metric().
89ebfbc
@patcon
Calculate utils.stats.two_prop_test() more effectively, and suppress …
2b9d845 
…divide-by-zero warning.
@patcon
Create comment projection functions. Add unit tests.
733b235
@patcon
Clean up variable names in comment projection functions.
b39dc1a
@patcon
Stop passing full pca object during comment projection.
5ba7356
@patcon
Rename more variables.
265fcf9
@patcon
Move pca unit tests.
7101f85
@patcon
Explain transpose a bit better.
f836796
@patcon
Add utils.run_pca() unit tests to compare against polismath results.
756b6d8
@patcon patcon force-pushed the more-comment-stats branch from 3d6ab89 to 756b6d8 Compare March 22, 2025 01:15
patcon added 14 commits March 21, 2025 21:20
@patcon
Copy link
Member Author

patcon commented Apr 6, 2025

this PR is pretty long-running, so going to just merge what's here, and create issues for the remaining things

@patcon patcon merged commit bb1e7cc into main Apr 6, 2025
2 checks passed
@patcon patcon deleted the more-comment-stats branch April 6, 2025 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@patcon