aai-institute / aai-institute/pyDVL

Refactoring of `EkfacInfluence`

Đang mở
#606 0 bình luận 0 reaction 1 người được giao Được @schroedk nhận Xem trên GitHub
breaking-change cleanup
Ngôn ngữ chính
Python
Star
146
Fork
10
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

By now, there is the inheritance `TorchInfluenceFunctionModel`->`EkfacInfluence`
and `EkfacInfluence` is the only existing subclass.

In order to reduce code duplication:

- Check, if it makes sense to inherit from `TorchComposableInfluence`
- If not, integrate functionality from base class `TorchInfluenceFunctionModel`
into `EkfacInfluence`
- Remove abstract base class `TorchInfluenceFunctionModel`

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Kiểm tra hệ thống phân cấp kế thừa giữa TorchInfluenceFunctionModel, TorchComposableInfluence và EkfacInfluence. Bắt đầu bằng cách đọc các định nghĩa lớp trong codebase để hiểu trách nhiệm hiện tại của chúng. Xác định xem EkfacInfluence có thể kế thừa từ TorchComposableInfluence để giảm sự trùng lặp hay việc hợp nhất TorchInfluenceFunctionModel vào EkfacInfluence sẽ phù hợp hơn. Chạy các bài kiểm thử hiện có cho EkfacInfluence để đảm bảo rằng việc tái cấu trúc không làm hỏng chức năng.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Lĩnh vực
devtools
Loại issue
Tái cấu trúc
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
45/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.