Some Client methods drop explicitly defined simvue args
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 52/100
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- Python
- Estrelas
- 4
- Forks
- 1
- Merge médio
- 22h 21min
- PRs com merge (30d)
- 6
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de simvue-io/python-api
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
simvue-io/python-api#994 · 1 comentário ·
-
bug
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 68/100
simvue-io/python-api#538 ·
-
bug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 55/100
simvue-io/python-api#1015 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 65/100
simvue-io/python-api#1000 · 2 comentários ·
-
bug
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
simvue-io/python-api#987 ·
Todas as issues de simvue-io/python-api
Issues semelhantes
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
aws-samples/sample-per-model-bedrock#261 ·
Mantenedores costumam responder em até 1 dia
-
[UI/Accessibility] Low contrast text visibility in country selection dropdown on settings pageAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
pwncollege/dojo#1188 ·
Mantenedores costumam responder em até 1 dia
-
bug launcher packaging
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
tenstorrent/tt-studio#1426 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
python-attrs/cattrs#796 ·
Mantenedores costumam responder em até 1 dia