Remove catchment_id from string construction

[cmaynard-ngwpc]

When creating the runtime error message for unit conversion updates in the NetCDFPerFeatureDataProvider, the format from the CSV error message was replicated, but the catchment_id was not available in the scope of this method. There’s not going to be a whole lot of different files, unlike potential different CSV files for different catchments so it is not necessary in this context. Stating NetCDFPerFeatureDataProvider is sufficient for the error message.

Removals

• Removed catchment_id from the model_name string construction.

Checklist

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

[cmaynard-ngwpc]

Closing this PR and using the updated version of PR 39 instead.