Correct inconsistent function name usage in code refactoring exercise

[matt-graham]

Fixes #466

Changes usages of compute_standard_deviation_by_data for function name to compute_standard_deviation_by_day to be consistent with earlier exercise. Between compute_standard_deviation_by_data and compute_standard_deviation_by_day the latter seemed the more descriptive of what the function computes and so the better to standardise on.

[github-actions]

Thank you!

Thank you for your pull request 😃

🤖 This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.

If you have files that automatically render output (e.g. R Markdown), then you should check for the following:

• 🎯 correct output
• 🖼️ correct figures
• ❓ new warnings
• ‼️ new errors

Rendered Changes

🔍 Inspect the changes: https://github.com/carpentries-incubator/python-intermediate-development/compare/md-outputs..md-outputs-PR-471

The following changes were observed in the rendered markdown documents:

 34-code-refactoring.md | 6 +++---
 md5sum.txt             | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

What does this mean?

If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible.

This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation.

⏱️ Updated at 2025-06-24 15:50:24 +0000

[smangham]

Good idea, and as we don't have any branches for 3.4+ in the example repo, this doesn't imply any extra work.