Skip to content

Fit latModel to actual BlueStore #16

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

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

Fit latModel to actual BlueStore #16

wants to merge 25 commits into from

Conversation

carlosmalt
Copy link
Owner

This pull request is to document fitting the radossim latModel to an actual BlueStore deployment in a Ceph cluster.

@carlosmalt
Copy link
Owner Author

Please don't merge this one.

@carlosmalt
Copy link
Owner Author

Now that we have K-S test scores the next step is to find an automatic way to fit the model across all sizes. Maybe using pypet?

@carlosmalt
Copy link
Owner Author

Excellent. The difference indeed shows it's all positive. I pushed a small change to skip the first 1000 measurements. One concern is the variance in transaction sizes. Can you compute the CDF and average of 10,000 transaction sizes for each request size (again skipping the first 1000)?

@yzhan298
Copy link
Collaborator

Excellent. The difference indeed shows it's all positive. I pushed a small change to skip the first 1000 measurements. One concern is the variance in transaction sizes. Can you compute the CDF and average of 10,000 transaction sizes for each request size (again skipping the first 1000)?

Sure, CDF should not be hard. actually for all graphs, I have skipped the first 1000 points, in the read_csv function(skiprows=1000).

@carlosmalt
Copy link
Owner Author

... actually for all graphs, I have skipped the first 1000 points, in the read_csv function (skiprows=1000).

Almost all -- check 0d5f9bf.

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.

2 participants