Make a central `PVValue` definition
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
- 35/100
Hướng nghiên cứu
Xác định mọi định nghĩa và import của PVValue, bao gồm cả những định nghĩa và import trong Genie Python và ibex-non-ca-helpers; trước tiên lập bản đồ các định nghĩa hiện tại và các nơi sử dụng chúng. Tập trung định nghĩa vào ibex-non-ca-helpers và thay thế các định nghĩa và import còn lại, sau đó xác minh rằng mọi cách sử dụng hiện tại đều được phân giải từ vị trí tập trung.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Feature Description
As a developer, I would like to be able to import PVValue from a central location anywhere in the project without needing to import Genie Python.
Explanation
Genie python is a large import, and so in some cases it has been avoided to import when just the PVValue type is required, this has lead to multiple definitions of PVValue in the project, these should be removed and replaced with a central definition in ibex-non-ca-helpers so that if needed the type can be modified in a single location, and code duplication is avoided.
Additional Information
As well as the changes in genie python and other locations where PVValue is defined, all imports of PVValue will need to be pointed at the new location.
Acceptance Criteria
-
PVValueis defined in a central location where it can easily be imported for all uses. - All current definitions of
PVValueand imports of them are replaced with an import of this central definition.
How to Review
Before making a PR...
- Provide verbose instructions for the reviewer to test your changes
- Describe how you have implemented testing for this feature
- Provide screenshots of the feature to help the reviewer if relevant
If not applicable, write "Not applicable"
...
To the reviewer: Make sure to update submodules!
- Ngôn ngữ chính
- Không có dữ liệu ngôn ngữ
- Star
- 6
- Fork
- 2
- Merge trung bình
- 12 ngày 19 giờ
- Pull request đã merge (30 ngày)
- 3
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 ISISComputingGroup/IBEX
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
ISISComputingGroup/IBEX#9023 ·
-
GALIL: test new amplifier PVsĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
ISISComputingGroup/IBEX#9005 ·
-
Good First Issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
ISISComputingGroup/IBEX#8889 ·
-
0
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 65/100
ISISComputingGroup/IBEX#2304 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
ISISComputingGroup/IBEX#2078 ·
Tất cả issue của ISISComputingGroup/IBEX
Issue tương tự
-
bug pkg: sphinx-needs
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
useblocks/sphinx-needs#2251 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
rhysd/actionlint#761 ·
-
Độ khó 2/5 Dưới một giờ Mức phù hợp với người mới 78/100
griffinwork40/agent-afk#3759 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug]: Importing weave changes the process-wide localeCó thể đã có người làm @himanshu748 đã nhận hôm nay. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày