Add benchmarks for geometry transformations (rot, flip, transpose) by sdebionne · Pull Request #525 · boostorg/gil

mloskot

approved these changes Oct 22, 2020

@mloskot

Both temp_red and max_color are float32_t so the result of the subraction already has the right type.
This should also remove the only dependency on Boost.NumericConversion

@mloskot

@mloskot

The "cxxstd" json field is being added to each Boost library's meta json information for libraries whose minumum C++ standard compilation level is C++11 on up. The value of this field matches one of the values for 'cxxstd' in Boost.Build. The purpose of doing this is to provide information for the Boost website documentation for each library which will specify the minimum C++ standard compilation that an end-user must employ in order to use the particular library. This will aid end-users who want to know if they can successfully use a Boost library based on their C++ compiler's  compilation level, without having to search the library's documentation to find this out. 

https://lists.boost.org/Archives/boost/2020/12/250543.php

@codejaeger

@mloskot

@sdebionne

@giomasce

@mloskot

…ts (boostorg#542)

If `Element` is a reference, then Element v{} is ill-formed.

Refines boostorg#273 which aimed to correctly value-initialize channel and pixel value members
The output type must be floating point, thus a check was added to make sure it
is the case. Though I had to add specific cases for float32_t as std::is_floating_point does not
consider it a floating point type

The accumulate part was wrong, it multiplied by delta_t on every sum, which is wrong

Use 8 way nabla compute.
This is just different discretization of Laplace operator
https://en.wikipedia.org/wiki/Discrete_Laplace_operator

Laplace stencils are now the same as in mathematical notation

The new function will provide a uniform way to generate stencils
by making sure directions are indexed properly

Add only required stencil points:
The 5 points Laplace stencil is now adding only required points and not assuming that others are zero

@mloskot

@mloskot

A new histogram class proposed with close suport for gil
image constructs.

Shift the stl support implmentation to extension to
serve as example for overloading fill_histogram.

Add cumulative histogram and histogram normalization.

Co-authored-by: debabrata1 <debabrata@goodhealthapp.com>
Co-authored-by: codejaeger <dhabalm1@>

@codejaeger

Allow non-equal dims along stack dim
Width can be different in hstack, height can be different in vstack

@codejaeger

Add basic GitHub Actions configuration based on mp11
Remove Actions jobs using GCC 4.7 and 4.8 - unsupported compilers
Run b2 with --abbreviate-paths on Windows
The -std=c++1z is broken for clang-4.0 but no need to test it
Add -mbig-obj to GCC on Windows
  - That is to avoid string table overflow and file too big
Define _GLIBCXX_USE_CXX11_ABI=0 for clang 3.5, 3.6, 3.7
  - Should help avoid linker error:
    `undefined reference to std::ios_base::failure::failure(char const*, std::error_code const&)`
Disable certain check in algorithm_channel_relation test for clang<3.8
Support construction from step_size, step_count, and a function for angles

Implement angle and radious version of Hough line transform and adds a demo
with static line that goes over secondary diagonal.

Implement incremental line raster
Implement naive line raster
Implement Bresenham line raster
Leave only Bresenham line rasterization

Naive and incremental algorithms were removed because they are supposed
to produce the same results anyway.
The reason for diverging results is inaccuracy of floating point numbers

Add circle rendering through trigonometric functions, using
arctan(1 / (radius + 1)) as minimal angle step.

Trigonometric circle rasterizer does not follow circle equation, but still
produces very round shapes.
A new testing methodology needs to be devised for this rasterizer.

The new version accepts start and points inclusively and tries to use
canonic representation during computations.

Slope decided to be is (diff_y + 1) / (diff_x + 1).

@mloskot

Most uses of cxxflags are wrong.
Use b2 warnings instead, e.g. warnings=pedantic

@mloskot

@mloskot

@mloskot

@mloskot

@marco-langer

This commit adds support for benchmark definition and running, currently supporting only cmake. Configuration was tested using GIL_USE_CONAN on, on Windows. One benchmark set was run successfully

@mloskot @sdebionne

@mloskot @sdebionne

The commit adds celero into conanfile and cmakelists, also provides one incorrectly implemented benchmark, which shows results that are completely inconsistent with google-benchmark
This commit adds a guide on how to write, build and execute benchmarks with supported frameworks (google-benchmark, celero). It also changes celero benchmark to crash instead of outputting wrong results.
This way we have a bit more control on release of dependencies what solves
some annoying issues (e.g. Conan is missing latest Celero fixed for VS2019).

@mloskot @sdebionne

Benchmark of various pixel iteration and access methods for
a contiguous images composed of homogeneous pixels with single
8-bit integer channel (grayscale).

@mloskot @sdebionne

@mloskot @sdebionne

@sdebionne

@sdebionne

@sdebionne

@sdebionne

@sdebionne

@sdebionne

@sdebionne

@sdebionne

@sdebionne