Skip to content

docs: correct two-phase-flow summary tables - #1264

Merged
jgostick merged 1 commit into
devfrom
docs/1229-two-phase-flow-summary-tables
Nov 12, 2026
Merged

jgostick merged 1 commit into
devfrom
docs/1229-two-phase-flow-summary-tables

Conversation

@jgostick

Copy link
Copy Markdown
Member

Render the three two-phase-flow summary tables correctly and align their phase labels, sentinels, and configuration masks with drainage, imbibition, and injection. Clarify the inverse-radius capillary-pressure relation and why injection pressure alone cannot reconstruct invasion history.

Validation: 409 unit tests passed, 3 skipped; all 51 notebook code cells passed with nbval-lax. Focused Sphinx/MyST-NB builds passed with warnings treated as errors, and HTML checks verified all three tables. Only four Markdown cells changed; notebook code, outputs, and metadata are preserved. The full test suite and full documentation site build were not run; nbval-lax does not compare saved outputs.

Original issue: #1229

@codecov

codecov Bot commented Nov 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.1%. Comparing base (4cad42c) to head (d573937).

Additional details and impacted files
@@          Coverage Diff          @@
##             dev   #1264   +/-   ##
=====================================
  Coverage   73.1%   73.1%           
=====================================
  Files         48      48           
  Lines       6976    6976           
=====================================
  Hits        5102    5102           
  Misses      1874    1874           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jgostick
jgostick marked this pull request as ready for review November 12, 2026 11:05
@jgostick
jgostick merged commit 20c6ad0 into dev Nov 12, 2026
21 checks passed
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.

1 participant