Skip to content

statistical testing - #139

Merged
DanielDeLayo merged 9 commits into
mainfrom
refactor
Feb 26, 2024
Merged

statistical testing#139
DanielDeLayo merged 9 commits into
mainfrom
refactor

Conversation

@DanielDeLayo

Copy link
Copy Markdown
Contributor

This tests the probability that a column will succeed given z non-zeros.
There is independence for a given z; we repeat for statistical significance.
There is likely not independence when varying z; for performance, the data point at z is based on the same state for the data point at z-1.

@etwest

Copy link
Copy Markdown
Collaborator

Another pull request deletes the statistical testing directory. I think this should just live in the tools directory.

@DanielDeLayo

Copy link
Copy Markdown
ContributorAuthor

What's the scope of the new repository? Should this live there instead?

@etwest

Copy link
Copy Markdown
Collaborator

What's the scope of the new repository? Should this live there instead?

The new repository is just for stream generation and parsing. I didn't move the other statistical tests there. I just deleted them.

@etwestetwest left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks overall good. Just add a little comment block at the top of both of these files to say what they do and we'll merge this in.

@etwestetwest left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yaaaaaaayyy....

@DanielDeLayo
DanielDeLayo merged commit b4aa8d5 into mainFeb 26, 2024
@DanielDeLayo
DanielDeLayo deleted the refactor branch February 26, 2024 17:21
Sign up for freeto 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.

2 participants

@DanielDeLayo@etwest