mirror of
https://github.com/boostorg/gil.git
synced 2026-01-25 06:12:22 +00:00
cf897c5969
* Simple implementation of lanczos scaling * Simple implementation of lanczos scaling * Refactor lanczos into separate header This commit moves the algorithm into its own header. It also provides templated interface now, utilizing a little bit of decltype and declval to resolve types for lambdas and stuff * Zero pixel at start of lanczos_at This commit fixes a possible bug and fixes some style incosistencies * Improve documentation for Lanczos This commit adds documentation which describes when to use the algorithm, a brief description of how it is supposed to work, and some caution on the quality of the output * Address style issues and fix warnings This commit fixes a style issue realted to namespace declaration and fixes a warning in added zeroing of a pixel at start of lanczos * text-realign function arguments * fix formatting issues * Implement handful of sanity tests This commit introduces a couple of sanity tests such as black image scaling to black image, and lanczos response being 0 at x = 0 * bracket on newline for for loops * add lanczos scaling to tests * more precision in lanczos calculation This commit migrates integral values to double precision and uses PI provided by boost.Math. These changes solve downscaled image being darker * Simple implementation of lanczos scaling * Refactor lanczos into separate header This commit moves the algorithm into its own header. It also provides templated interface now, utilizing a little bit of decltype and declval to resolve types for lambdas and stuff * Zero pixel at start of lanczos_at This commit fixes a possible bug and fixes some style incosistencies * Improve documentation for Lanczos This commit adds documentation which describes when to use the algorithm, a brief description of how it is supposed to work, and some caution on the quality of the output * Address style issues and fix warnings This commit fixes a style issue realted to namespace declaration and fixes a warning in added zeroing of a pixel at start of lanczos * text-realign function arguments * fix formatting issues * Implement handful of sanity tests This commit introduces a couple of sanity tests such as black image scaling to black image, and lanczos response being 0 at x = 0 * bracket on newline for for loops * add lanczos scaling to tests * more precision in lanczos calculation This commit migrates integral values to double precision and uses PI provided by boost.Math. These changes solve downscaled image being darker * Add Jamfile for ip test directory Add Jamfile for image_processing test directory and build-project from outer test directory * Add IP test directory to ci build This commit adds a line at the end of .ci/build-and-test.sh to include image processing tests in CI builds * Remove redundant lines from Jamfile Simplify Jamfile at test/image_processing * Rewrite range condition x > -a && x < a exchange with -a < x && x < a in lanczos * Add newline at the end of files * Add math and lexical_cast to get-boost As math and lexical_cast are used in lanczos scaling, both were added to get-boost * Revert "Add newline at the end of files" Since Boost.Math.Constants is overengineered for use case by requiring Boost.lexical_cast or a global define, it is removed This reverts commit 0743ab072ff3455421853697f7f46aee7d22382c. * Define pi and use in lanczos Value of pi is moved out into detail and used by lanczos * Move image_processing to test/core * formatting fixes * Adjust CMakeLists for moved IP tests This commit removes image_processing as subdirectory from test/ and adds to test/core's CMakeLists * Remove unused from get-boost Since lanczos scaling no longer uses boost.math.constants, lexical_cast has been removed as well * Downgrade math to transitive dep This is a stray change left from my incorrect resetting of the HEAD on lanczos branch * Remove unnecessary includes io includes are not used in lanczos scaling test, and they break build * Fix ambiguous overload issue for min Since width() and height() now return std::ptrdiff_t, call to std::min needs a cast. There was also shadowing of pi declared in detail/math.hpp, which is also fixed * Apply mloskot's patch The patch provided by Mateusz changes all usages of long int into ptrdiff_t, which is returned by width() and height() functions of image_view * Apply .editorconfig rules This commit is a simple reformat of affected files * Use aliases x_coord_t and y_coord_t The change converges integer handling in arguments, using view' type aliases. View arguments have to come first to avoid non-deduced context problem. Also replaced long int with ptrdiff_t in lanczos, in numeric.hpp file * Apply alias usage for rest of the code Some places with ptrdiff_t were left out from previous commit, so they are changed in this one. Plus a fix for max call being ambiguous. * Replace all literals with casted vars This commit takes extreme stance of never using an integer literal due to problems in ambiguity of deduction of min and max functions * Fix unenclosed foreach During rebase of gsoc2019 onto develop, I didn't enclose first foreach with matching endforeach, hence tests failed
66 lines
1.5 KiB
C++
Executable File
66 lines
1.5 KiB
C++
Executable File
#include <boost/gil/image_processing/scaling.hpp>
|
|
#include <boost/gil/image.hpp>
|
|
|
|
#include <boost/core/lightweight_test.hpp>
|
|
|
|
|
|
#include <iostream>
|
|
|
|
namespace gil = boost::gil;
|
|
|
|
bool are_equal(gil::rgb8_view_t expected, gil::rgb8_view_t actual) {
|
|
if (expected.dimensions() != actual.dimensions())
|
|
return false;
|
|
|
|
for (long int y = 0; y < expected.height(); ++y)
|
|
{
|
|
for (long int x = 0; x < expected.width(); ++x)
|
|
{
|
|
if (expected(x, y) != actual(x, y))
|
|
{
|
|
return false;
|
|
}
|
|
}
|
|
}
|
|
|
|
return true;
|
|
}
|
|
|
|
void test_lanczos_black_image()
|
|
{
|
|
|
|
const gil::point_t input_dimensions(20, 20);
|
|
const gil::point_t output_dimensions(input_dimensions.x / 2, input_dimensions.y / 2);
|
|
gil::rgb8_image_t image(input_dimensions, gil::rgb8_pixel_t(0, 0, 0), 0);
|
|
// fill with values other than 0
|
|
gil::rgb8_image_t output_image(
|
|
output_dimensions,
|
|
gil::rgb8_pixel_t(100, 100, 100),
|
|
0
|
|
);
|
|
gil::rgb8_image_t expected(
|
|
output_dimensions,
|
|
gil::rgb8_pixel_t(0, 0, 0),
|
|
0
|
|
);
|
|
|
|
auto view = gil::view(image);
|
|
auto output_view = gil::view(output_image);
|
|
auto expected_view = gil::view(expected);
|
|
gil::scale_lanczos(view, output_view, 5);
|
|
BOOST_TEST(are_equal(expected_view,output_view));
|
|
}
|
|
|
|
void test_lanczos_response_on_zero()
|
|
{
|
|
//random value for a
|
|
BOOST_TEST(gil::lanczos(0, 2) == 1);
|
|
}
|
|
|
|
int main()
|
|
{
|
|
test_lanczos_black_image();
|
|
test_lanczos_response_on_zero();
|
|
return boost::report_errors();
|
|
}
|