fix: remove unused argument from load_GDP

[ollie-bell]

Closes # (if applicable).

Changes proposed in this Pull Request

Remove unused countries_codes argument from load_GDP function in build_shapes.py script.

Checklist

• I consent to the release of this PR's code under the AGPLv3 license and non-code contributions under CC0-1.0 and CC-BY-4.0.
• I tested my contribution locally and it seems to work fine.
• Code and workflow changes are sufficiently documented.
• Newly introduced dependencies are added to envs/environment.yaml and doc/requirements.txt.
• Changes in configuration options are added in all of config.default.yaml and config.tutorial.yaml.
• Add a test config or line additions to test/ (note tests are changing the config.tutorial.yaml)
• Changes in configuration options are also documented in doc/configtables/*.csv and line references are adjusted in doc/configuration.rst and doc/tutorial.rst.
• A note for the release notes doc/release_notes.rst is amended in the format of previous release notes, including reference to the requested PR.

[ollie-bell]

This unused argument was not being considered when calling the function. Because these are passed as positional arguments, they were not being passed as intended (e.g. update was taking the value of out_logging)

pypsa-earth/scripts/build_shapes.py

Line 697 in 62691b6

GDP_tif, name_tif = load_GDP(year, update, out_logging, name_file_nc)

[davide-f]

Many thanks @ollie-bell !
Would you like to add a minor release note? :)

[ollie-bell]

Many thanks @ollie-bell ! Would you like to add a minor release note? :)

Done!

[davide-f]

Welcome among contributors @ollie-bell :D