Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Isolate pass service errors & potentially log and discard invalid proposal info

Abierto
  • ¿Algo no está bien?
#71 0 comentarios 0 reacciones 1 asignado Ver en GitHub

@stuartcampbell ya está trabajando en esto.

Desde el 11/5/2024.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

Consider breaking up the try-catch blocks here (and similar uses elsewhere) to:

  1. Isolate the source of an error
  2. Possibly log and then discard invalid proposal types

https://github.com/NSLS2/nsls2api/blob/a2de1f81dbc5b9cef843826e3c7b917ba77d6f58/src/nsls2api/services/pass_service.py#L75-L88

For example:

 try: 
     pass_proposal_types_list = await _call_pass_webservice(url) 
 except Exception as error: 
     error_message = "Error retrieving proposal types from PASS." 
     logger.exception(error_message) 
     raise PassException(error_message) from error 

proposal_types = [] 
if pass_proposal_types_list and len(pass_proposal_types_list) > 0: 
    for proposal_type in pass_proposal_types_list: 
        try: 
            proposal_types.append(PassProposalType(**proposal_type)) 
        except ValidationError as error: 
            error_message = f"Error validating data recevied from PASS for proposal type for the {facility} facility." 
            logger.error(error_message) 
            # raise PassException(error_message) from error
Lenguaje dominante
Python
Estrellas
0
Forks
13
Sin PR fusionados en 30 d

Preparar el entorno

  • Incluye un Dockerfile o un archivo de Docker Compose
  • Sin plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de NSLS2/nsls2api

Todos los issues de NSLS2/nsls2api

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.