Decouple CompoundGenerator behaviours
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
Hướng nghiên cứu
Bắt đầu bằng cách đọc CompoundGenerator cùng với các generator Zip và Concat được nêu trong issue, sau đó theo dõi cách việc lồng nhau và tạo point hiện đang tương tác với nhau. Xác định mối tương ứng với các khái niệm Generator được mô tả cho Python và Java trước khi quyết định có thể tách các trách nhiệm như thế nào. Được xem là hoàn tất khi CompoundGenerator hoạt động như một Generator thông thường, có thể được bọc bởi Zip hoặc Concat, và các model có thể được populate mà không cần generator bọc bên ngoài.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
CompoundGenerator performs 2 functions: nesting generators and creating all the points for a scan.
With the addition of Zip and Concat generators, the nesting behaviour should be removed from the the creating points generators, as these generators must work on Generators (which a CompoundGenerator is not). These models also hint that there could be further ways of combining two generators (although I'm struggling to think of geometries other than a inside b, a alongside b, a after b)
This would allow CompoundGenerator to become another normal Generator, which can be wrapped by a Zip/Concat, and could allow a model to be populated with points without a wrapping generator, which would make the matching of Java and Python concepts of generators in GDA a little simpler.
Cons: pretty large rework
Pros: separation of concerns, allows nesting Compounded models within other 'Multimodels'
- Ngôn ngữ chính
- Python
- Star
- 6
- Fork
- 6
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
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 DiamondLightSource/scanpointgenerator
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 68/100
-
ROIExcluder rename, ROIIncluderĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
-
Update Jython travis buildĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 30/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 30/100
Tất cả issue của DiamondLightSource/scanpointgenerator
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/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 84/100
spec-kitty/spec-kitty#5319 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
backend::vllm diffusion multimodal
Độ 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 84/100
openai/openai-agents-python#5229 ·
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 84/100
Maintainer thường phản hồi trong vòng 1 ngày