Update Heat Tutorials, fix typos.

[LScheib]

Due Diligence

• General:
  • base branch must be main for new features, latest release branch (e.g. release/1.3.x) for bug fixes
  • title of the PR is suitable to appear in the Release Notes
• Implementation:
  • unit tests: all split configurations tested
  • unit tests: multiple dtypes tested
  • documentation updated where needed

Description

Issue resolved: #1235
To get started with Heat I went through the documentation and particularly the provided tutorials. While doing so, I noticed a few minor linguistic mistakes and typos in code snippets, which I correct here. Also I try to slightly improve the uniformity of the given code examples.

Does this change modify the behaviour of other functions? If so, which?

no

[ghost]

👇 Click on the image for a new way to code review

Legend

[github-actions]

Thank you for the PR!

[codecov]

Codecov Report

Merging #1238 (368646e) into main (5beb254) will increase coverage by 0.57%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

❗ Current head 368646e differs from pull request most recent head 6468616. Consider uploading reports for the commit 6468616 to get more accurate results

@@            Coverage Diff             @@
##             main    #1238      +/-   ##
==========================================
+ Coverage   91.75%   92.32%   +0.57%     
==========================================
  Files          77       77              
  Lines       11080    11056      -24     
==========================================
+ Hits        10166    10207      +41     
+ Misses        914      849      -65     

Flag	Coverage Δ
unit	92.32% <ø> (+0.57%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 8 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[mrfh92]

Looks fine to me.
Thanks 👍

[mrfh92]

CI seems to get stuck at "ci/codebase Expected — Waiting for status to be reported"...

[mrfh92]

CI gets stuck again... this is strange since, e.g., #1236 did not get stuck

[mrfh92]

@mtar when youre fit again, can you have a look on this?

[ClaudiaComito]

I think the pipeline isn't triggering because Receive PR isn't triggered by changes in doc/ (for good reasons). We should merge anyway.

[ClaudiaComito]

Thanks a lot @LScheib !!