Skip to content

Fix tiny bug printing the index of the experiment in the idealised flow example#57

Merged
michaeldenes merged 2 commits intomainfrom
fix_print_experiment_name
Sep 25, 2024
Merged

Fix tiny bug printing the index of the experiment in the idealised flow example#57
michaeldenes merged 2 commits intomainfrom
fix_print_experiment_name

Conversation

@michaeldenes
Copy link
Member

This PR is to fix a tiny printing issue in the idealised flow example. The final block of code plots the positions of the particles and prints the depth range of the particles for each experiment. The plots have 'Experiment i+1', but the print statements show 'experiment i'.

@michaeldenes michaeldenes marked this pull request as ready for review September 25, 2024 13:21
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.66%. Comparing base (a61baa1) to head (d6a41d8).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #57   +/-   ##
=======================================
  Coverage   66.66%   66.66%           
=======================================
  Files           8        8           
  Lines         717      717           
=======================================
  Hits          478      478           
  Misses        239      239           
Flag Coverage Δ
unit-tests 66.66% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@michaeldenes michaeldenes merged commit 757234e into main Sep 25, 2024
@michaeldenes michaeldenes deleted the fix_print_experiment_name branch September 25, 2024 13:47
fanqi203 pushed a commit to fanqi203/plasticparcels that referenced this pull request Jul 8, 2025
…ment_name

Fix tiny bug printing the index of the experiment in the idealised flow example
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