Skip to content

fix: MSE→RMSE terminology consistency and classification report table alignment#960

Merged
leestott merged 2 commits intomainfrom
copilot/fix-code-for-review-comments
Apr 11, 2026
Merged

fix: MSE→RMSE terminology consistency and classification report table alignment#960
leestott merged 2 commits intomainfrom
copilot/fix-code-for-review-comments

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 11, 2026

Two doc correctness fixes in response to review thread comments on PR #951.

Changes

2-Regression/3-Linear/README.md

  • Updated prose from "mean square error (MSE)" → "root mean square error (RMSE)" with corrected definition ("root of the mean…")
  • Renamed variable msermse; updated print label from 'Mean error:''RMSE:'
# Before
mse = np.sqrt(mean_squared_error(y_test, pred))
print(f'Mean error: {mse:3.3} ({mse/np.mean(pred)*100:3.3}%)')

# After
rmse = np.sqrt(mean_squared_error(y_test, pred))
print(f'RMSE: {rmse:3.3} ({rmse/np.mean(pred)*100:3.3}%)')

4-Classification/2-Classifiers-1/README.md

  • Fixed accuracy row column alignment in classification report table — values were under precision/recall; moved to f1-score/support to match actual sklearn.classification_report() output
precision recall f1-score support
accuracy 0.80 1199 0.80 1199

@leestott leestott marked this pull request as ready for review April 11, 2026 21:43
Copilot AI changed the title [WIP] Fix code for all comments in review thread fix: MSE→RMSE terminology consistency and classification report table alignment Apr 11, 2026
Copilot AI requested a review from leestott April 11, 2026 21:45
@leestott leestott merged commit fb665b5 into main Apr 11, 2026
1 check 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.

2 participants