-
Notifications
You must be signed in to change notification settings - Fork 7
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
Added the bounds on the logarithmic batch size variables. #39
Conversation
The black format check failed. @AlbertLee125 |
Those bounds seem not only arbitrary but extremely loose (remember that is the logarithm of the volume!). Try running FBBT to figure out reasonable bounds if not provided |
I ran the fbbt and set up the new bounds for the batch_size_log. |
This is the output of the logramithic size of the batch. I think I can set up the bounds of the Logarithmic Batch Size of the Products from 0 to 10. |
I have tightened this bound. |
Hi @AlbertLee125 . We can run |
Running the fbbt for 200 times I think setting the bounds from 0 to 10 is the best I can do. |
Fixes #30
I managed an issue batch_processing benchmark #30 by adding bounds on the variable.
The issue is commented at
#30