Some Client methods drop explicitly defined simvue args
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 52/100
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- Python
- Étoiles
- 4
- Forks
- 1
- Merge moyen
- 22 h 21 min
- PR mergées (30 j)
- 6
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de simvue-io/python-api
-
bug
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
simvue-io/python-api#538 ·
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
simvue-io/python-api#1015 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
simvue-io/python-api#1000 · 2 commentaires ·
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
simvue-io/python-api#987 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
simvue-io/python-api#932 ·
Toutes les issues de simvue-io/python-api
Issues similaires
-
Difficulté 1/5 1-3 heures Accessibilité débutants 68/100
FavioVazquez/showtime#101 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
rossant/awesome-math#199 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
nautechsystems/nautilus_trader#5365 ·
Les mainteneurs répondent en général sous 1 jour
-
Examples say cnxOuvertedocumentation embedded
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour