Merge pull request #266 from ecmwf-projects/feature/extra-exception

ecmwf-cobarzan · web-flow · commit abe91fc8b08b · 2025-01-31T14:06:13.000Z
Feature/extra exception
diff --git a/cads_adaptors/tools/area_selector.py b/cads_adaptors/tools/area_selector.py
@@ -8,7 +8,7 @@
 from earthkit.transforms import tools as eka_tools
 
 from cads_adaptors.adaptors import Context
-from cads_adaptors.exceptions import InvalidRequest
+from cads_adaptors.exceptions import CdsFormatConversionError, InvalidRequest
 from cads_adaptors.tools import adaptor_tools, convertors
 
 
@@ -312,7 +312,7 @@ def area_selector_paths(
                 out_paths += area_selector_path(
                     path, area=area, context=context, **kwargs
                 )
-            except NotImplementedError:
+            except (NotImplementedError, CdsFormatConversionError):
                 context.logger.debug(
                     f"could not convert {path} to xarray; returning the original data"
                 )

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`from earthkit.transforms import tools as eka_tools`
`9`	`9`
`10`	`10`	`from cads_adaptors.adaptors import Context`
`11`		`-from cads_adaptors.exceptions import InvalidRequest`
	`11`	`+from cads_adaptors.exceptions import CdsFormatConversionError, InvalidRequest`
`12`	`12`	`from cads_adaptors.tools import adaptor_tools, convertors`
`13`	`13`
`14`	`14`
`@@ -312,7 +312,7 @@ def area_selector_paths(`
`312`	`312`	`out_paths += area_selector_path(`
`313`	`313`	`path, area=area, context=context, **kwargs`
`314`	`314`	`)`
`315`		`- except NotImplementedError:`
	`315`	`+ except (NotImplementedError, CdsFormatConversionError):`
`316`	`316`	`context.logger.debug(`
`317`	`317`	`f"could not convert {path} to xarray; returning the original data"`
`318`	`318`	`)`