Tiny edit to allow target naming templating to be used

[norlandrhagen]

Change Summary

While using funnel for caching, I ran into an unexpected error when trying to apply formatting for target names based on prefect parameters.

traceback: ```└── 16:41:35 | ERROR | Unexpected error: KeyError('gcm') Traceback (most recent call last): File "/Users/nrhagen/opt/anaconda3/envs/install/envs/cmip6-jupyter/lib/python3.9/site-packages/prefect/engine/runner.py", line 48, in inner new_state = method(self, state, *args, **kwargs) File "/Users/nrhagen/opt/anaconda3/envs/install/envs/cmip6-jupyter/lib/python3.9/site-packages/prefect/engine/task_runner.py", line 926, in get_task_run_state result = self.result.write(value, **formatting_kwargs) File "/Users/nrhagen/opt/anaconda3/envs/install/envs/cmip6-jupyter/lib/python3.9/site-packages/funnel/prefect/result.py", line 72, in write new = self.format(**{}) File "/Users/nrhagen/opt/anaconda3/envs/install/envs/cmip6-jupyter/lib/python3.9/site-packages/prefect/engine/result/base.py", line 133, in format new.location = new.location.format(**kwargs) KeyError: 'gcm' └── 16:41:35 | INFO | Task 'test_kwargs': Finished task run for task with final state: 'Failed' └── 16:41:35 | INFO | Flow run FAILED: some reference tasks failed. Flow run failed!

By editing line 72 of funnel/prefect/result.py from new = self.format(**{}) to new = self.format(**kwargs), the targeting templating seemed to succeed.

Related issue number

Checklist

• Unit tests for the changes exist
• Tests pass on CI
• Documentation reflects the changes where applicable

PS This package has been incredibly useful, thanks for all the work developing it.

[andersy005]

Thank you for putting this together, @norlandrhagen! We recently decided to move the caching functionality from funnel into a standalone package: https://github.com/NCAR/xpersist (ncar-xdev/xpersist#35). Do you mind opening the PR over there instead?