Some Client methods drop explicitly defined simvue args
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 52/100
Línea de trabajo
Start by comparing the credential forwarding in SimvueObject.get at simvue/api/objects/base.py:536-544 with Artifact.get and Artifact.from_run in simvue/api/objects/artifact/fetch.py, then inspect SimvueObject.__init__ at base.py:249. Trace Client.get_folder through get_folder_from_path in simvue/api/objects/folder.py as another example. Done means checking the object methods and ensuring explicitly supplied server URL and token reach each yielded object.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description of Bug
simvue/api/objects/artifact/fetch.py:168 (Artifact.get) and :331 (Artifact.from_run) yield:
But SimvueObject.__init__ unconditionally calls SimvueConfiguration.fetch() at construction — so every yielded artifact re-derives config from the process env and raises.
https://github.com/simvue-io/python-api/blob/027bcba8a79d0850d96397df51e3aa806836346a/simvue/api/objects/base.py#L249
Note the contrast: the inherited SimvueObject.get does forward server_url/server_token into the objects it yields — that's why runs/folders/metrics all work and only artifacts fail.
https://github.com/simvue-io/python-api/blob/027bcba8a79d0850d96397df51e3aa806836346a/simvue/api/objects/base.py#L536-L544
The hand-written Artifact.get/from_run just missed it. Client.list_artifacts (client.py:512) forwards credentials for the HTTP list call, but they're lost on the yielded objects.
Client.get_folder → get_folder_from_path (api/objects/folder.py) → Folder.get(filters=[...], count=1) also drops server_url/server_token.
https://github.com/simvue-io/python-api/blob/027bcba8a79d0850d96397df51e3aa806836346a/simvue/api/objects/folder.py#L371
I haven't checked all files, these are just the ones I happened to stumble across. Need to go through and check that each object passes through url and token correctly
- Lenguaje dominante
- Python
- Estrellas
- 4
- Forks
- 1
- Merge medio
- 22 h 21 min
- PR fusionados (30 d)
- 6
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 simvue-io/python-api
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
simvue-io/python-api#994 ·
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
simvue-io/python-api#538 ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
simvue-io/python-api#1015 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
simvue-io/python-api#1000 · 2 comentarios ·
-
bug
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
simvue-io/python-api#987 ·
Todos los issues de simvue-io/python-api
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
huggingface/sentence-transformers#4186 ·
Los mantenedores suelen responder en 5 días
-
documentation good first issue
Dificultad 2/5 Medio día Aptitud para principiantes 72/100
sujithputta02/Esapay#13 ·
-
bot-triaged module: correctness (silent) module: mps module: NaNs and Infs triage review
Dificultad 2/5 Medio día Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
README: CI link/badge checkAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
SCIInstitute/SCIRun#2842 ·
Los mantenedores suelen responder en 3 días