From c74d22aa4208d7dc5541e820f9b12b2f3eeec137 Mon Sep 17 00:00:00 2001
From: Sean Sube <seansube@gmail.com>
Date: Fri, 17 Feb 2023 07:49:45 -0600
Subject: [PATCH] fix(api): continue converting other models after an error in
 one (#166)

---
 api/onnx_web/convert/__main__.py | 64 +++++++++++++++++++-------------
 1 file changed, 39 insertions(+), 25 deletions(-)

diff --git a/api/onnx_web/convert/__main__.py b/api/onnx_web/convert/__main__.py
index a6901616..ea62fd6b 100644
--- a/api/onnx_web/convert/__main__.py
+++ b/api/onnx_web/convert/__main__.py
@@ -194,8 +194,12 @@ def convert_models(ctx: ConversionContext, args, models: Models):
             else:
                 model_format = source_format(model)
                 source = model["source"]
-                dest = fetch_model(ctx, name, source, model_format=model_format)
-                logger.info("finished downloading source: %s -> %s", source, dest)
+
+                try:
+                    dest = fetch_model(ctx, name, source, model_format=model_format)
+                    logger.info("finished downloading source: %s -> %s", source, dest)
+                except Exception as e:
+                    logger.error("error fetching source %s: %s", name, e)
 
     if args.diffusion and "diffusion" in models:
         for model in models.get("diffusion"):
@@ -206,23 +210,27 @@ def convert_models(ctx: ConversionContext, args, models: Models):
                 logger.info("skipping model: %s", name)
             else:
                 model_format = source_format(model)
-                source = fetch_model(
-                    ctx, name, model["source"], model_format=model_format
-                )
 
-                if model_format in model_formats_original:
-                    convert_diffusion_original(
-                        ctx,
-                        model,
-                        source,
-                    )
-                else:
-                    convert_diffusion_stable(
-                        ctx,
-                        model,
-                        source,
+                try:
+                    source = fetch_model(
+                        ctx, name, model["source"], model_format=model_format
                     )
 
+                    if model_format in model_formats_original:
+                        convert_diffusion_original(
+                            ctx,
+                            model,
+                            source,
+                        )
+                    else:
+                        convert_diffusion_stable(
+                            ctx,
+                            model,
+                            source,
+                        )
+                except Exception as e:
+                    logger.error("error converting diffusion model %s: %s", name, e)
+
     if args.upscaling and "upscaling" in models:
         for model in models.get("upscaling"):
             model = tuple_to_upscaling(model)
@@ -232,10 +240,14 @@ def convert_models(ctx: ConversionContext, args, models: Models):
                 logger.info("skipping model: %s", name)
             else:
                 model_format = source_format(model)
-                source = fetch_model(
-                    ctx, name, model["source"], model_format=model_format
-                )
-                convert_upscale_resrgan(ctx, model, source)
+
+                try:
+                    source = fetch_model(
+                        ctx, name, model["source"], model_format=model_format
+                    )
+                    convert_upscale_resrgan(ctx, model, source)
+                except Exception as e:
+                    logger.error("error converting upscaling model %s: %s", name, e)
 
     if args.correction and "correction" in models:
         for model in models.get("correction"):
@@ -246,11 +258,13 @@ def convert_models(ctx: ConversionContext, args, models: Models):
                 logger.info("skipping model: %s", name)
             else:
                 model_format = source_format(model)
-                source = fetch_model(
-                    ctx, name, model["source"], model_format=model_format
-                )
-                convert_correction_gfpgan(ctx, model, source)
-
+                try:
+                    source = fetch_model(
+                        ctx, name, model["source"], model_format=model_format
+                    )
+                    convert_correction_gfpgan(ctx, model, source)
+                except Exception as e:
+                    logger.error("error converting correction model %s: %s", name, e)
 
 def main() -> int:
     parser = ArgumentParser(