-
Notifications
You must be signed in to change notification settings - Fork 35
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
Performance impact #24
Comments
Hi, Thanks! The performance impact depends on the processing cost of a single iteration. Please note that the progress is reported to the log and to the web client every 5 seconds, so having too many iterations per sec. does not make much sense. If this is a case, I would recommand dividing the computation in batches and measuring the progress of batch completions. Best, |
Hi, thanks for your answer, I understand your idea of using batches, will try to see how it can fit nicely in the code (ideally adding a function on BaseModel to make it reusable). |
Surprising. 70 iterations per sec should not be a big deal. What is your Odoo version? |
Odoo v14 |
Hi,
Thanks for this module, it is a really good enhancement, and i'm using it often now.
However, I have noticed that using this progressbar seems to have some impacts on the performance/speed of the process.
For example, without progressbar, the progress takes around 30 seconds, but with the progressbar, it will take 1min30.
I am the only one to notice this ? Is it expected ?
Thanks
The text was updated successfully, but these errors were encountered: