Skip to content

🚀 feat(stones): Enhance stone laying rate calculation with habitat factor#1956

Merged
mkmccarty merged 1 commit intomainfrom
mm-branch-1
Dec 16, 2025
Merged

🚀 feat(stones): Enhance stone laying rate calculation with habitat factor#1956
mkmccarty merged 1 commit intomainfrom
mm-branch-1

Conversation

@mkmccarty
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings December 16, 2025 22:36
@mkmccarty mkmccarty merged commit 47c1ed0 into main Dec 16, 2025
8 checks passed
@mkmccarty mkmccarty deleted the mm-branch-1 branch December 16, 2025 22:36
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes an inconsistency in stone laying rate calculations by adding the habitat factor (collegHab) to the second calculation loop. Previously, the first calculation loop (line 852) included the habitat factor while the second loop (line 870) did not, resulting in inconsistent calculations.

Key Changes

  • Added * collegHab multiplication to the stone laying rate calculation in the details/solo data loop to match the calculation in the optimization loop

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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