Rename 'struct bmi' to not conflict with declarations from BMI headers

[PhilMiller]

In service of NOAA-OWP/ngen#831, I encountered a name conflict between struct bmi here and namespace bmi from the BMI C++ header:

[ 31%] Building CXX object CMakeFiles/sft_pframework.dir/src/main_pseudo_framework.cxx.o
In file included from /home/runner/work/SoilFreezeThaw/SoilFreezeThaw/src/main_pseudo_framework.cxx:13:
/home/runner/work/SoilFreezeThaw/SoilFreezeThaw/extern/evapotranspiration/include/pet.h:214:8: error: ‘struct bmi’ redeclared as different kind of entity
  214 | struct bmi
      |        ^~~
In file included from /home/runner/work/SoilFreezeThaw/SoilFreezeThaw/src/main_pseudo_framework.cxx:5:
/home/runner/work/SoilFreezeThaw/SoilFreezeThaw/src/../bmi/bmi.hxx:13:11: note: previous declaration ‘namespace bmi { }’
   13 | namespace bmi {
      |           ^~~

We should not have things named simply bmi to avoid conflicts like this.

Changes

• Rename struct bmi to struct pet_bmi and change the one instance of usage

Testing

1. CI

Checklist

• [x] PR has an informative and human-readable title
• [x] Changes are limited to a single goal (no scope creep)
• [x] Code can be automatically merged (no conflicts)
• [x] Code follows project standards (link if applicable)
• [x] Passes all existing automated tests
• [x] Any change in functionality is tested
• [x] New functions are documented (with a description, list of inputs, and expected output)
• [x] Placeholder code is flagged / future todos are captured in comments
• [x] Visually tested in supported browsers and devices (see checklist below :point_down:)
• [x] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
• [x] Reviewers requested with the Reviewers tool :arrow_right:

[PhilMiller]

I can't directly add reviewers due too permissions issues, but @aaraney @SnowHydrology @madMatchstick could one of you please approve and merge this? I don't think it will impact anything outside this repo, so no coordination needed.