Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for wider channel size in f3d::image #858

Merged
merged 24 commits into from
Jul 10, 2023
Merged

Support for wider channel size in f3d::image #858

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
b805150
Support for wider channel size in f3d::image
Meakk Jun 16, 2023
80dc774
store internal image using vtkImageData
Meakk Jun 19, 2023
878f1f9
remove useless init
Meakk Jun 19, 2023
ca5fd59
fix python
Meakk Jun 20, 2023
2faf297
style
Meakk Jun 20, 2023
631b83b
deprecated
Meakk Jun 21, 2023
c305fe8
ci
Meakk Jun 21, 2023
c1c1e14
ci
Meakk Jun 21, 2023
8970b61
cov
Meakk Jun 23, 2023
3149100
cppcheck
Meakk Jun 23, 2023
cdd52de
exr
Meakk Jun 29, 2023
f4d849a
fix
Meakk Jun 29, 2023
d872de9
doc
Meakk Jun 29, 2023
2c0928a
review
Meakk Jun 29, 2023
a27f11f
ci
Meakk Jun 29, 2023
07165d4
review
Meakk Jul 10, 2023
6e79864
python
Meakk Jul 10, 2023
4409185
fix
Meakk Jul 10, 2023
8899676
style
Meakk Jul 10, 2023
9de666e
style
Meakk Jul 10, 2023
15ccc58
fix deprecated python
Meakk Jul 10, 2023
2e7afa3
style
Meakk Jul 10, 2023
a84339d
review mw
Meakk Jul 10, 2023
d766c80
style
Meakk Jul 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
style
  • Loading branch information
@Meakk
Meakk committed Jul 10, 2023
commit 88996767a06ad37d130a119eed777f50f5657e05
12 changes: 8 additions & 4 deletions python/F3DPythonBindings.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,8 @@ PYBIND11_MODULE(f3d, module)
.value("FLOAT", f3d::image::ChannelType::FLOAT)
.export_values();
Meakk marked this conversation as resolved.
Show resolved Hide resolved

auto setImageBytes = [](f3d::image& img, const py::bytes& data) {
auto setImageBytes = [](f3d::image& img, const py::bytes& data)
{
const py::buffer_info info(py::buffer(data).request());
size_t expectedSize =
img.getChannelCount() * img.getWidth() * img.getHeight() * img.getChannelTypeSize();
Expand All @@ -94,7 +95,8 @@ PYBIND11_MODULE(f3d, module)
img.setContent(info.ptr);
};

auto getImageBytes = [](const f3d::image& img) {
auto getImageBytes = [](const f3d::image& img)
{
size_t expectedSize =
img.getChannelCount() * img.getWidth() * img.getHeight() * img.getChannelTypeSize();
return py::bytes(static_cast<char*>(img.getContent()), expectedSize);
Expand Down Expand Up @@ -258,13 +260,15 @@ PYBIND11_MODULE(f3d, module)

// deprecated functions, will be removed in the next major release
Meakk marked this conversation as resolved.
Show resolved Hide resolved
#ifndef F3D_NO_DEPRECATED
image.def("setResolution", [](f3d::image& img, unsigned int width, unsigned height) {
image.def("setResolution", [](f3d::image& img, unsigned int width, unsigned height)
{
PyErr_WarnEx(PyExc_DeprecationWarning,
"setResolution is deprecated, use the appropriate constructor instead.", 1);
return img.setResolution(width, height);
});

image.def("setChannelCount", [](f3d::image& img, unsigned int channels) {
image.def("setChannelCount", [](f3d::image& img, unsigned int channels)
{
PyErr_WarnEx(PyExc_DeprecationWarning,
"setChannelCount is deprecated, use the appropriate constructor instead.", 1);
return img.setChannelCount(channels);
Expand Down