Repository navigation
test: validate pinned consumer recovery and publication in CI - #37
Open
BaruqueRodrigues wants to merge 5 commits into
Open
BaruqueRodrigues wants to merge 5 commits into
BaruqueRodrigues wants to merge 5 commits into
Conversation
esloch
approved these changes
Oct 28, 2026
esloch
left a comment
Collaborator
There was a problem hiding this comment.
A correção da perda dos parâmetros na recuperação municipal resolve o problema que identificamos anteriormente. A preservação da precisão numérica na geração SQL também é importante.
A ressalva continua sendo a validação com INLA real, principalmente quanto a concorrência, memória e falhas operacionais, conforme solicitado no PR #34.
Lembrando que ele depende do #36.
added 5 commits
October 29, 2026 14:31
BaruqueRodrigues
force-pushed
the
codex/pr10-epidemiological-equivalence
branch
from
November 3, 2026 15:51
1534d94 to
cdec43b
Compare
BaruqueRodrigues
force-pushed
the
codex/pr11-consumer-integration
branch
from
November 3, 2026 15:51
15e0915 to
b600216
Compare
esloch
approved these changes
Nov 5, 2026
esloch
left a comment
Collaborator
There was a problem hiding this comment.
Thanks, @BaruqueRodrigues.
I reviewed the latest changes.
The remaining validation should focus on the full production workload, using our current baseline of approximately 2h40 on 8 cores, along with the staging Zika audit and scientific review.
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
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.
Execute the pinned principal consumer's state execution, municipal recovery and publication contracts against controlled inputs. Cover failure isolation, all-city failure, dengue/chikungunya/Zika, synthetic RData normalization, numeric SQL precision and idempotent PostgreSQL persistence. Preserve immutable consumer/reference sources and compare persisted content by natural keys.
Reapply on the corrected stack, removing the duplicate parameter helper already moved to #28. Add real Bayesian consumer recovery: the mandatory INLA runner now requires all four scenarios and fails on skips, missing scenarios, warnings or errors. Upload comparison outputs, source manifests and environment evidence.
Validation already performed locally on the consolidated candidate: PostgreSQL suite passed 474 expectations with zero failures/warnings (four INLA-only skips executed separately). The mandatory real-INLA suite passed all four scenarios and 14 expectations on both macOS and Docker, without skips/failures/warnings. Docker R CMD check with built vignettes passed with zero errors/warnings and one documented global-connection NOTE; static lint passed. Evidence and reproduction details are committed under docs/refactoring/evidence/review-2026-10-29/.
Operational gates remain open: real staging Zika coverage requires read-only access and an operational municipality list, neither present in the repository. No consumer pin, migration registry or production database was changed. National capacity and posterior intervals across different fitting-thread budgets require operational/scientific assessment. This PR does not declare production migration complete.
Stacked on #36; #20 remains the integration PR pending review and explicit merges.