Skip to content

Conversation

d-morrison
Copy link
Owner

No description provided.

@Copilot Copilot AI review requested due to automatic review settings July 28, 2025 18:33
Copy link
Contributor

@Copilot 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 a mathematical notation error by removing an incorrect double accent in the MLE (Maximum Likelihood Estimator) notation. The change corrects the notation from $\hat{\vb}$ to $\hat{\beta}$ for consistency with standard mathematical conventions.

Key changes:

  • Corrected LaTeX notation for MLE parameter from double-accented to single-accented format

@@ -7,7 +7,7 @@ differentiate or difference $\red{\eta(\vx)}$ with respect to $\red{x_j}$ (depen

$$\deriv{\red{x_j}} \red{\eta(\vx)}$$

**In order to find the MLE $\hat{\vb}$**:
**In order to find the MLE $\hat{\beta}$**:
Copy link
Preview

Copilot AI Jul 28, 2025

Choose a reason for hiding this comment

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

The notation change from \hat{\vb} to \hat{\beta} appears inconsistent with the vector notation \vb used in line 12. If \vb represents a vector parameter, the MLE should maintain the vector notation as \hat{\mathbf{\beta}} or similar to preserve consistency.

Suggested change
**In order to find the MLE $\hat{\beta}$**:
**In order to find the MLE $\hat{\vb}$**:

Copilot uses AI. Check for mistakes.

Copy link

PR Preview Action v1.6.2

🚀 View preview at
https://d-morrison.github.io/rme/pr-preview/pr-207/

Built to branch gh-pages at 2025-07-28 19:51 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@d-morrison
Copy link
Owner Author

the double accent looks bad, but it's correct; instead of this change, I'm going to change my vector notation to undertilde, boldface, or both.

@d-morrison d-morrison closed this Sep 2, 2025
@d-morrison d-morrison deleted the remove-double-accent branch September 2, 2025 19:29
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