AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Remove Imath and Pystring from nanoColor

Đang mở
#2,002 3 bình luận 0 reaction 0 người được giao Xem trên GitHub
Feature Request
Ngôn ngữ chính
C++
Star
2.1k
Fork
503
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

One of the requirements for nanoColor is that it have no external dependencies. Most of the dependencies have already been removed, via optional CMake flags. However, code from Imath and Pystring was temporarily moved to the ext folder (i.e. the required code from those projects was copied in, strictly as a temporary prototyping measure).

For Pystring, the proposal is to reimplement any functions currently needed by OCIO (there are not many).

For Imath, the only thing that is used by OCIO is the half class. The proposal is that a half class be created specifically for OCIO, based on the Imath code but with some simplifications. Given that most hardware these days has specific instructions for half-floats, those would be used if available. For other machines, the code from Imath that does not use the LUT would be used.

For simplicity, these changes would apply to both nanoColor and full OCIO. If people would like to continue to be able to use the full Imath or Pystring as an optional dependency with full OCIO, please note that as a comment below.

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

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

Hướng nghiên cứu

Bắt đầu bằng việc kiểm tra thư mục ext và các cờ CMake hiện đang khiến Imath và Pystring trở thành tùy chọn hoặc được nhúng. Truy vết các hàm Pystring và lớp half của Imath được nanoColor và OCIO đầy đủ sử dụng; hoàn thành khi cả hai bản build hoạt động mà không cần các dependency bên ngoài và các dependency tùy chọn của thư viện đầy đủ đã được xử lý khi cần.

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

Đánh giá

Công nghệ
cmake, cpp
Lĩnh vực
build-system, tooling
Loại issue
Tái cấu trúc
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
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
25/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.