avoid failure when docstrings have been stripped (python -OO)

JohannesBuchner · JohannesBuchner · commit 30d621e15cc6 · 2025-07-14T15:21:16.000+02:00
diff --git a/keras/src/applications/densenet.py b/keras/src/applications/densenet.py
@@ -487,6 +487,6 @@ def decode_predictions(preds, top=5):
     A Keras model instance.
 """
 
-setattr(DenseNet121, "__doc__", DenseNet121.__doc__ + DOC)
-setattr(DenseNet169, "__doc__", DenseNet169.__doc__ + DOC)
-setattr(DenseNet201, "__doc__", DenseNet201.__doc__ + DOC)
+setattr(DenseNet121, "__doc__", (DenseNet121.__doc__ or '') + DOC)
+setattr(DenseNet169, "__doc__", (DenseNet169.__doc__ or '') + DOC)
+setattr(DenseNet201, "__doc__", (DenseNet201.__doc__ or '') + DOC)
diff --git a/keras/src/applications/resnet.py b/keras/src/applications/resnet.py
@@ -586,6 +586,7 @@ def decode_predictions(preds, top=5):
     A Model instance.
 """
 
-setattr(ResNet50, "__doc__", ResNet50.__doc__ + DOC)
-setattr(ResNet101, "__doc__", ResNet101.__doc__ + DOC)
-setattr(ResNet152, "__doc__", ResNet152.__doc__ + DOC)
+if ResNet50.__doc__ is not None:
+    setattr(ResNet50, "__doc__", ResNet50.__doc__ + DOC)
+    setattr(ResNet101, "__doc__", ResNet101.__doc__ + DOC)
+    setattr(ResNet152, "__doc__", ResNet152.__doc__ + DOC)
diff --git a/keras/src/applications/resnet_v2.py b/keras/src/applications/resnet_v2.py
@@ -203,6 +203,7 @@ def decode_predictions(preds, top=5):
     A Model instance.
 """
 
-setattr(ResNet50V2, "__doc__", ResNet50V2.__doc__ + DOC)
-setattr(ResNet101V2, "__doc__", ResNet101V2.__doc__ + DOC)
-setattr(ResNet152V2, "__doc__", ResNet152V2.__doc__ + DOC)
+if ResNet50V2.__doc__ is not None:
+    setattr(ResNet50V2, "__doc__", ResNet50V2.__doc__ + DOC)
+    setattr(ResNet101V2, "__doc__", ResNet101V2.__doc__ + DOC)
+    setattr(ResNet152V2, "__doc__", ResNet152V2.__doc__ + DOC)
diff --git a/keras/src/optimizers/adadelta.py b/keras/src/optimizers/adadelta.py
@@ -130,6 +130,7 @@ def get_config(self):
         return config
 
 
-Adadelta.__doc__ = Adadelta.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if Adadelta.__doc__ is not None:
+    Adadelta.__doc__ = Adadelta.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/adafactor.py b/keras/src/optimizers/adafactor.py
@@ -209,6 +209,7 @@ def get_config(self):
         return config
 
 
-Adafactor.__doc__ = Adafactor.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if Adafactor.__doc__ is not None:
+    Adafactor.__doc__ = Adafactor.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/adagrad.py b/keras/src/optimizers/adagrad.py
@@ -103,6 +103,7 @@ def get_config(self):
         return config
 
 
-Adagrad.__doc__ = Adagrad.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if Adagrad.__doc__ is not None:
+    Adagrad.__doc__ = Adagrad.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/adam.py b/keras/src/optimizers/adam.py
@@ -148,7 +148,7 @@ def get_config(self):
         )
         return config
 
-
-Adam.__doc__ = Adam.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if Adam.__doc__ is not None:
+    Adam.__doc__ = Adam.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/adamax.py b/keras/src/optimizers/adamax.py
@@ -141,6 +141,7 @@ def get_config(self):
         return config
 
 
-Adamax.__doc__ = Adamax.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if Adamax.__doc__ is not None:
+    Adamax.__doc__ = Adamax.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/adamw.py b/keras/src/optimizers/adamw.py
@@ -95,6 +95,7 @@ def __init__(
             )
 
 
-AdamW.__doc__ = AdamW.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if AdamW.__doc__ is not None:
+    AdamW.__doc__ = AdamW.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/ftrl.py b/keras/src/optimizers/ftrl.py
@@ -233,7 +233,7 @@ def get_config(self):
         )
         return config
 
-
-Ftrl.__doc__ = Ftrl.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if Ftrl.__doc__ is not None:
+    Ftrl.__doc__ = Ftrl.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/lamb.py b/keras/src/optimizers/lamb.py
@@ -143,6 +143,7 @@ def get_config(self):
         return config
 
 
-Lamb.__doc__ = Lamb.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if Lamb.__doc__ is not None:
+    Lamb.__doc__ = Lamb.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/lion.py b/keras/src/optimizers/lion.py
@@ -131,6 +131,7 @@ def get_config(self):
         return config
 
 
-Lion.__doc__ = Lion.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if Lion.__doc__ is not None:
+    Lion.__doc__ = Lion.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/loss_scale_optimizer.py b/keras/src/optimizers/loss_scale_optimizer.py
@@ -307,6 +307,7 @@ def from_config(cls, config, custom_objects=None):
         return cls(inner_optimizer, **config)
 
 
-LossScaleOptimizer.__doc__ = LossScaleOptimizer.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if LossScaleOptimizer.__doc__ is not None:
+    LossScaleOptimizer.__doc__ = LossScaleOptimizer.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/nadam.py b/keras/src/optimizers/nadam.py
@@ -158,6 +158,7 @@ def get_config(self):
         return config
 
 
-Nadam.__doc__ = Nadam.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if Nadam.__doc__ is not None:
+    Nadam.__doc__ = Nadam.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/rmsprop.py b/keras/src/optimizers/rmsprop.py
@@ -167,6 +167,7 @@ def get_config(self):
         return config
 
 
-RMSprop.__doc__ = RMSprop.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if RMSprop.__doc__ is not None:
+    RMSprop.__doc__ = RMSprop.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )
diff --git a/keras/src/optimizers/sgd.py b/keras/src/optimizers/sgd.py
@@ -133,6 +133,7 @@ def get_config(self):
         return config
 
 
-SGD.__doc__ = SGD.__doc__.replace(
-    "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
-)
+if SGD.__doc__ is not None:
+    SGD.__doc__ = SGD.__doc__.replace(
+        "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args
+    )

Original file line number	Diff line number	Diff line change
`@@ -95,6 +95,7 @@ def __init__(`
`95`	`95`	`)`
`96`	`96`
`97`	`97`
`98`		`-AdamW.__doc__ = AdamW.__doc__.replace(`
`99`		`- "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args`
`100`		`-)`
	`98`	`+if AdamW.__doc__ is not None:`
	`99`	`+ AdamW.__doc__ = AdamW.__doc__.replace(`
	`100`	`+ "{{base_optimizer_keyword_args}}", optimizer.base_optimizer_keyword_args`
	`101`	`+ )`