Skip to content

Docs update model install instructions#3769

Merged
infotroph merged 7 commits intoPecanProject:developfrom
man080107:docs-update-model-install-instructions
Mar 1, 2026
Merged

Docs update model install instructions#3769
infotroph merged 7 commits intoPecanProject:developfrom
man080107:docs-update-model-install-instructions

Conversation

@man080107
Copy link
Copy Markdown
Contributor

Description

Motivation and Context

Review Time Estimate

  • Immediately
  • Within one week
  • When possible

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation.
  • My name is in the list of CITATION.cff
  • I agree that PEcAn Project may distribute my contribution under any or all of
    • the same license as the existing code,
    • and/or the BSD 3-clause license.
  • I have updated the CHANGELOG.md.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

#Fixes #3741

Copy link
Copy Markdown
Member

@infotroph infotroph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apparently I started reviewing this some time ago and never posted my comments. I think this is a solid improvement that needs a bit more iteration -- see my inline comments for details.

Comment thread book_source/03_topical_pages/03_pecan_xml.Rmd Outdated
Comment thread book_source/03_topical_pages/03_pecan_xml.Rmd Outdated
Comment thread book_source/03_topical_pages/03_pecan_xml.Rmd Outdated
Expanded the section on meteorological inputs to include details about the <met> tag and its sub-tags, along with an example XML structure.
@man080107
Copy link
Copy Markdown
Contributor Author

Thank you for the thoughtful review and the inline comments. I’m glad to hear this is a solid improvement. I’ll address the suggested changes and push an updated version shortly..

Comment thread book_source/03_topical_pages/12_troubleshooting-pecan.Rmd Outdated
@infotroph infotroph enabled auto-merge March 1, 2026 11:56
@infotroph infotroph added this pull request to the merge queue Mar 1, 2026
Merged via the queue into PecanProject:develop with commit 9712d31 Mar 1, 2026
19 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Restructure deeply nested documentation sections to improve navigation

2 participants