{{ message }}
fix: ETA calculation returning zero prematurely due to precision rounding#23
Open
insign wants to merge 1 commit into
Open
fix: ETA calculation returning zero prematurely due to precision rounding#23insign wants to merge 1 commit into
insign wants to merge 1 commit into
Conversation
O método `getOverallEstimatedTimeRemaining` retornava o tempo restante como zero antes da conclusão real se o progresso arredondado (`$overallProgress >= 100`) atingisse 100%, o que pode acontecer dependendo da precisão configurada (ex: um processo em 100% e outro em 99.4% arredonda para 100% caso `precision = 0`). O patch altera a condição de retorno antecipado para `if ($this->isOverallComplete())`, garantindo que todos os processos estejam em 100%. A taxa de tempo para o overall agora usa a média sem arredondamentos para não perder a precisão nos cálculos, e ambas as funções de ETA (`getEstimatedTimeRemaining` e `getOverallEstimatedTimeRemaining`) agora usam `ceil()` ao invés de `round()` para evitar que o tempo caia de 1 para 0 prematuramente (antes do fim real). O commit também conta com testes de regressão de borda. Co-authored-by: insign <1113045+insign@users.noreply.github.com>
Contributor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Este PR corrige um bug onde o tempo estimado de conclusão global (
getOverallEstimatedTimeRemaining) poderia retornar 0 (zero) prematuramente devido ao arredondamento da variável$overallProgress. Ele garante cálculos de ETA confiáveis e exatos baseados na integridade real do progresso de todos os processos subjacentes.Ao invés de verificar se o progresso arredondado atingiu 100%, o código agora utiliza corretamente
$this->isOverallComplete()para verificar se todos os workers reportaram de fato a conclusão total. Ademais, o progresso médio não arredondado passou a ser usado no cálculo da taxa de tempo, e as funções de ETA passaram a usarceil()para não zerar o ETA com base no valor arredondado (quando restar 0,4 segundos por exemplo). A correção inclui o teste de regressão para assegurar a confiabilidade.PR created automatically by Jules for task 17118859250131402652 started by @insign