regrid.py: Partials

[mxkpp]

Closing earlier PR #86 for this one.

This PR includes commits from #98 so it could be merged on its own (larger PR) or after #98 (for smaller diff).

Steps taken for this PR:

1. Add new Partials class to regrid.py to store partial functions which significantly reduce parameter passing and line count.

2. Replace existing partials calls with usage of the new Partials class.

3. Use regex to replace existing log calls with the new Partials calls.

Regex replacements:

### Debug True, ending with comma and line break
err_handler\.log_msg\(\s*config_options,\s*mpi_config,\s*debug=True,\s*msg=(.*?),\n
pt.log_debug(msg=$1

### Debug True, ending with close parens
err_handler\.log_msg\(\s*config_options,\s*mpi_config,\s*debug=True,\s*msg=(.*?)\)
pt.log_debug(msg=$1)

### Debug True, ending with raw line break
err_handler\.log_msg\(\s*config_options,\s*mpi_config,\s*debug=True,\s*msg=(.*?)\n
pt.log_debug(msg=$1


### Info (debug omitted), ending with comma and line break
err_handler\.log_msg\(\s*config_options,\s*mpi_config,\s*msg=(.*?),\n
pt.log_info(msg=$1

### Info (debug omitted), ending with close parens
err_handler\.log_msg\(\s*config_options,\s*mpi_config,\s*msg=(.*?)\)
pt.log_info(msg=$1)

### Info (debug omitted), ending with raw line break
err_handler\.log_msg\(\s*config_options,\s*mpi_config,\s*msg=(.*?)\n
pt.log_info(msg=$1


### Warning, ending with comma and line break
err_handler\.log_warning\(\s*config_options,\s*mpi_config,\s*msg=(.*?),\n
pt.log_warn(msg=$1

### Warning, ending with close parens
err_handler\.log_warning\(\s*config_options,\s*mpi_config,\s*msg=(.*?)\)
pt.log_warn(msg=$1)

### Warning, ending with raw line break
err_handler\.log_warning\(\s*config_options,\s*mpi_config,\s*msg=(.*?)\n
pt.log_warn(msg=$1


### Error, ending with comma and line break
err_handler\.log_error\(\s*config_options,\s*mpi_config,\s*msg=(.*?),\n
pt.log_err(msg=$1

### Error, ending with close parens
err_handler\.log_error\(\s*config_options,\s*mpi_config,\s*msg=(.*?)\)
pt.log_err(msg=$1)

### Error, ending with raw line break
err_handler\.log_error\(\s*config_options,\s*mpi_config,\s*msg=(.*?)\n
pt.log_err(msg=$1


### Critical, ending with comma and line break
err_handler\.log_critical\(\s*config_options,\s*mpi_config,\s*msg=(.*?),\n
pt.log_crit(msg=$1

### Critical, ending with close parens
err_handler\.log_critical\(\s*config_options,\s*mpi_config,\s*msg=(.*?)\)
pt.log_crit(msg=$1)

### Critical, ending with raw line break
err_handler\.log_critical\(\s*config_options,\s*mpi_config,\s*msg=(.*?)\n
pt.log_crit(msg=$1 

Additions

class Partials in regrid.py

Removals

• Previous log calls

Changes

• Replace log calls with usage of Partials

Testing

1. Tested calibrations and forecasts in RTE

Screenshots

Notes

Todos

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
• Visually tested in supported browsers and devices (see checklist below 👇)
• Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
• Reviewers requested with the Reviewers tool ➡️

Testing checklist

Target Environment support

• Linux

[mdeshotel]

Can we be more verbose and use args1 args2. Or maybe even args_esmf args_log.

[mxkpp]

Updated to args1 and args2, keeping it nameless since each tuple might serve multiple purposes.

[mdeshotel]

I think we can drop the msg= for these method calls. I think it is pretty clear what we are passing in and I think positionally we are fine. Please double check before making the change.

[mxkpp]

Updated to remove the instances of msg=