Infinite recursion error on deep copy `AzureBackendConfig` due to __getattr__ implementation
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Comienza en azure-quantum/azure/quantum/qiskit/backends/backend.py, en AzureBackendConfig.getattr, y luego ejecuta la reproducción de deepcopy del issue. Verifica que copiar una instancia con metadatos vacíos ya no provoque RecursionError, mientras que los atributos ausentes sigan provocando AttributeError y los atributos respaldados por metadatos sigan funcionando.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The class AzureBackendConfig in azure.quantum.qiskit.backends.backend has a field metadata containing any remaining properties of the class. The __getattr__ method of this class is implemented in such a way that, if the attribute is not found, the metadata dict is checked. However, this leads to a RecursionError: maximum recursion depth exceeded when an instance of the class is copied using copy.deepcopy (which apparently happens sometimes).
class AzureBackendConfig:
...
def __getattr__(self, name: str) -> Any:
if name == "max_experiments":
return 1
try:
return self.__dict__[name]
except KeyError as exc:
if name in self.metadata:
return self.metadata[name]
raise AttributeError(
f"'{type(self).__name__}' object has no attribute '{name}'"
) from exc
Simplified reproduction of the error:
import copy
from dataclasses import dataclass, field
from typing import Any
@dataclass
class A:
metadata: dict[str, Any] = field(default_factory=dict)
def __getattr__(self, name: str) -> Any:
try:
return self.__dict__[name]
except KeyError as exc:
if name in self.metadata:
return self.metadata[name]
raise AttributeError(
f"'{type(self).__name__}' object has no attribute '{name}'"
) from exc
a = A()
copy.deepcopy(a) # RecursionError: maximum recursion depth exceeded
A possible solution would be to change the problematic line to:
if "metadata" in self.__dict__ and name in self.metadata:
- Lenguaje dominante
- Python
- Estrellas
- 161
- Forks
- 113
- Merge medio
- 4 d 21 h
- PR fusionados (30 d)
- 3
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de microsoft/azure-quantum-python
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
-
Export typesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 55/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
microsoft/azure-quantum-python#650 · 1 comentario ·
-
azure-quantum
Dificultad 2/5 1-3 horas Aptitud para principiantes 20/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
microsoft/azure-quantum-python#416 · 3 comentarios ·
Todos los issues de microsoft/azure-quantum-python
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
modelcontextprotocol/python-sdk#3648 ·
Los mantenedores suelen responder en 1 día
-
docs good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
VenetoStato/giorgio#6 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
Claiming namespace ddalusAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 70/100
EclipseFdn/open-vsx.org#13831 ·
Los mantenedores suelen responder en 1 día