Skip to content

minor fixes: moving a formula and updating outdated references #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Oct 23, 2024

Conversation

mr-threepwood
Copy link

The placement of the formula at the beginning of chapter 12.1. seems a little off for me in regard to ending the first paragraph by "As you might remember from school this is simply:" and being immediately followed by the next paragraph.

magnitude_old

I would suggest moving the formula up accordingly.

magnitude_new

@mr-threepwood mr-threepwood requested a review from a team as a code owner October 17, 2024 02:25
@mr-threepwood
Copy link
Author

Added a quick fix for failing instruction cargo run --bin magnitude in analogy to pull request #6.

fix to the fix: reference path updated

another fix to the fix: reference updated
@mr-threepwood
Copy link
Author

Updated an outdated reference in the section explore referring to the snake game which had become chapter 14 in this edition of the book.

I apologize for the many commits here in this PR and even force pushing after doing a rebase to squash silly "fixes of the fix" as I hit commit too soon and too often before setting up the link to the snake game chapter correctly. 😓

@mr-threepwood mr-threepwood changed the title readability improvement: move magnetic field magnitude formula up by one paragraph minor fixes: moving a formula and updating outdated references Oct 17, 2024
@mr-threepwood
Copy link
Author

Found and updated another reference in explore.md to match the current chapter numbers.

@mr-threepwood
Copy link
Author

One more oudated reference name found in appendix/3-mag-calibration and updated.

@BartMassey
Copy link
Member

Thank you for these! Apologies for the slow response: was out of country last week.

@BartMassey BartMassey merged commit 43b9f45 into rust-embedded:main Oct 23, 2024
3 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.

2 participants