Update nwp-consumer container tag to 1.1.32

[peterdudfield]

Pull Request

Description

• India NWP consumer up to 1.1.32, this is needed for metoffice change
• ECWMF: Increase memory and CPU for ECWMF consumer
• GFS: Add higher tolerence for nans,

How Has This Been Tested?

• manually put in on dev overnight

Checklist:

• My code follows OCF's coding style guidelines
• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works
• I have checked my code and corrected any misspellings

[devsjc]

By doing the class parameters like this you're losing any kind of hinting you might get from your IDE on what the ContainerDefinition class is expecting, which will increase the likelihood of erroneus entries. I'm not sure it's necessarily a better or clearer way of assigning the containers.

Either way, don't forget to run uv run ruff format src/airflow_dags/dags/india/consume-nwp-dag.py to fix up the spacing in those dicts.

[peterdudfield]

uv run ruff format src/airflow_dags/dags/india/consume-nwp-dag.py

Ill do the formatting,

Should i just do two different containers?

[devsjc]

I guess in this case it depends what the more useful information for a developer reading this is: either, what each container's environment individually is (in which case do two containers), or how the containers' environments differ (in which case keep as is). It's probably only really me and you looking at this, so happy to go with you if you reckon parsing their differences more quickly is of greater value. It's not super important!

[peterdudfield]

I guess in this case it depends what the more useful information for a developer reading this is: either, what each container's environment individually is (in which case do two containers), or how the containers' environments differ (in which case keep as is). It's probably only really me and you looking at this, so happy to go with you if you reckon parsing their differences more quickly is of greater value. It's not super important!

Thanks, ill keep it as it is at the moment. Two containers looked a bit too much like duplicate code