Isolate pass service errors & potentially log and discard invalid proposal info
@stuartcampbell ci sta già lavorando.
Dal 11/5/2024.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Consider breaking up the try-catch blocks here (and similar uses elsewhere) to:
- Isolate the source of an error
- Possibly log and then discard invalid proposal types
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
- Lingua principale
- Python
- Stelle
- 0
- Fork
- 13
- Merge medio
- 1h 17m
- PR unite (30g)
- 1
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di NSLS2/nsls2api
-
Review outstanding TODOs Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
Tutte le issue di NSLS2/nsls2api
Issue simili
-
agent-ready documentation needs-triage
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
-
instance instance add
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
searxng/searx-instances#939 · 1 commento ·
-
area-deployment area-integrations triage:bot-seen
Difficoltà 2/5 Mezza giornata Idoneità per principianti 86/100