Skip to content

Address review suggestions for equation docs and method counts - #39

Merged
cursor[bot] merged 3 commits into
mainfrom
cursor/review-suggestions-doc-fixes-b57b
Jul 12, 2026
Merged

Address review suggestions for equation docs and method counts#39
cursor[bot] merged 3 commits into
mainfrom
cursor/review-suggestions-doc-fixes-b57b

Conversation

@brunomartinsmv

Copy link
Copy Markdown
Owner

Aplica as sugestões de revisão levantadas no PR de documentação de equações (v2.0.0). Este PR é apenas documentação — não inclui bump de versão nem itens de release.

Alterações

Contagem de métodos (15 → 18)

  • docs/methodology.md e README.md agora refletem 18 métodos alternativos + Penman-Monteith como referência, alinhado a configs/methods.yml (15 computed + 3 precomputed_only).
  • A seção "Computed 15-method comparison scope" no README permanece correta para o escopo calculado pelo pipeline.

Jensen-Heise: comportamento abaixo do limiar

  • docs/equations/et0_methods.md e docs/methodology.md documentam explicitamente que jensen_heise não aplica clamp em (T_mean - T_x) — valores negativos são possíveis quando T_mean < T_x.

Ortografia PT

  • docs/equations/et0_methods.md: "Calibracao" → "Calibração".

Limpeza de backlog

  • tasks/todo.md: removida entrada [x] duplicada sobre derivações LaTeX.

Nota sobre escopo de release

A sugestão de separar bump de versão (pyproject.toml, CITATION.cff, CHANGELOG) em PR próprio permanece válida para releases futuros; este PR não altera artefatos de versão.

Open in Web Open in Cursor 

- Update alternative method count from 15 to 18 in methodology and README
- Document Jensen-Heise lack of temperature clamp in equations and methodology
- Fix PT spelling: Calibracao -> Calibração in et0_methods.md
- Remove duplicate completed entry in tasks/todo.md

Co-authored-by: Bruno <brunomartinsmv@users.noreply.github.com>
@cursor
cursor Bot marked this pull request as ready for review July 12, 2026 21:49
cursoragent and others added 2 commits July 12, 2026 21:49
Co-authored-by: Bruno <brunomartinsmv@users.noreply.github.com>
…igured

Co-authored-by: Bruno <brunomartinsmv@users.noreply.github.com>
@cursor
cursor Bot merged commit 46f8e0b into main Jul 12, 2026
4 checks passed
@cursor
cursor Bot deleted the cursor/review-suggestions-doc-fixes-b57b branch July 12, 2026 21:52
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