Some Client methods drop explicitly defined simvue args
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 52/100
Hướng nghiên cứu
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.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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
- Ngôn ngữ chính
- Python
- Star
- 4
- Fork
- 1
- Merge trung bình
- 22 giờ 21 phút
- Pull request đã merge (30 ngày)
- 6
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của simvue-io/python-api
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
simvue-io/python-api#994 ·
-
bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 68/100
simvue-io/python-api#538 ·
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
simvue-io/python-api#1015 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 65/100
simvue-io/python-api#1000 · 2 bình luận ·
-
bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
simvue-io/python-api#987 ·
Tất cả issue của simvue-io/python-api
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
jbacalso24/mimry#291 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
good first issue
Độ khó 2/5 Dưới một giờ Mức phù hợp với người mới 82/100
eli-alvarado/us-watchlist-alerts#14 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
from-review-extraction python severity:nit
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 80/100
LearningCircuit/local-deep-research#7462 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
mermaid_extract.py silently discards single-line state-diagram notes; sequence notes are keptĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
cathrynlavery/diagram-design#441 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
[Cleanup] Include the public header in work_split.cppCó thể đã có người làm @blozano-tt đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
tenstorrent/tt-metal#60850 ·
Maintainer thường phản hồi trong vòng 1 ngày