Up to standards ✅

_{TIP This summary will be updated as you push new changes. Give us feedback}

Thank you! Will be a nice improvement

Just a few to-dos here:

• no need to import DataFrame, since we typically just use pd.DataFrame
• no need to import List, since we use the nice typing introduced by Python 3.10
• you don't need to wrap the outputs into a list, just having both outputs in the same return line will output a tuple, which is better
• You'll need to update the test_get_biodiversity unit tests in test_core_functions to deal with this changed output format, so that the tests will run through successfully before I merge:-)

Pythonic tuple construction works like (pd.DataFrame, pd.DataFrame) (in theory you can do tuple((pd.DataFrame, pd.DataFrame)) but it's a bit cumbersome:-)
I'd recommend running the tests locally until they run through (or at least try to run the biodiversity unit test in isolation until it passes). But we should be almost there I think

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.69%. Comparing base (7ca4484) to head (34c1666).
⚠️ Report is 10 commits behind head on dev.

@@           Coverage Diff           @@
##              dev     #114   +/-   ##
=======================================
  Coverage   96.69%   96.69%           
=======================================
  Files          18       18           
  Lines        7264     7266    +2     
=======================================
+ Hits         7024     7026    +2     
  Misses        240      240           

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