Age | Commit message (Collapse) | Author | |
---|---|---|---|
2020-08-07 | Cleanup: Compositor, Clang-Tidy else-after-return fixes | Sybren A. Stüvel | |
This addresses warnings from Clang-Tidy's `readability-else-after-return` rule in the `source/blender/compositor` module. No functional changes. | |||
2020-05-08 | Cleanup: take includes out of 'extern "C"' blocks | Jacques Lucke | |
Surrounding includes with an 'extern "C"' block is not necessary anymore. Also that made it harder to add any C++ code to some headers, or include headers that have "optional" C++ code like `MEM_guardedalloc.h`. I tested compilation on linux and windows (and got help from @LazyDodo). If this still breaks compilation due to some linker error, the header containing the symbol in question is probably missing an 'extern "C"' block. Differential Revision: https://developer.blender.org/D7653 | |||
2019-04-23 | Cleanup: style, use braces for compositor | Campbell Barton | |
2019-04-17 | ClangFormat: apply to source, most of intern | Campbell Barton | |
Apply clang format as proposed in T53211. For details on usage and instructions for migrating branches without conflicts, see: https://wiki.blender.org/wiki/Tools/ClangFormat | |||
2019-02-18 | Cleanup: conform headers to have license first | Campbell Barton | |
Also remove doxy comments for licenses and add missing GPL header. | |||
2019-02-01 | Cleanup: remove redundant, invalid info from headers | Campbell Barton | |
BF-admins agree to remove header information that isn't useful, to reduce noise. - BEGIN/END license blocks Developers should add non license comments as separate comment blocks. No need for separator text. - Contributors This is often invalid, outdated or misleading especially when splitting files. It's more useful to git-blame to find out who has developed the code. See P901 for script to perform these edits. | |||
2018-11-14 | Cleanup: comment block tabs | Campbell Barton | |
2018-06-29 | Cleanup: trailing newlines | Campbell Barton | |
2018-06-17 | Cleanup: trailing space for compositor | Campbell Barton | |
2018-06-01 | Cleanup: trailing whitespace (comment blocks) | Campbell Barton | |
Strip unindented comment blocks - mainly headers to avoid conflicts. | |||
2015-10-10 | Fix various compiler warnings. | Brecht Van Lommel | |
2015-03-27 | Compositor: Code cleanup, prepare for strict C++ flags | Sergey Sharybin | |
2015-01-19 | D627: Memory usage optimization for the compositor. | Jeroen Bakker | |
The compostor used a fixed size of 4 floats to hold pixel data. this patch will select size of a pixel based on its type. It uses 1 float for Value, 3 float for vector and 4 floats for color data types. When benchmarking on shots (opening shot of caminandes) we get a reduction of memory of 30% and a tiny speedup as less data transformations needs to take place (but these are negligable. More information of the patch can be found on https://developer.blender.org/D627 and http://wiki.blender.org/index.php/Dev:Ref/Proposals/Compositor2014_p1.1_TD Developers: jbakker & mdewanchand Thanks for Sergey for his indept review. | |||
2014-06-01 | Fix T40459: Gauss table can be NULL when ending the blur node operation, | Lukas Tönne | |
which must not be passed to MEM_freeN. | |||
2014-04-15 | Structural cleanup and improvements for the compositor. | Lukas Tönne | |
Many parts of the compositor are unnecessarily complicated. This patch aims at reducing the complexity of writing nodes and making the code more transparent. == Separating Nodes and Operations == Currently these are both mixed in the same graph, even though they have very different purposes and are used at distinct stages in the compositing process. The patch introduces dedicated graph classes for nodes and for operations. This removes the need for a lot of special case checks (isOperation etc.) and explicit type casts. It simplifies the code since it becomes clear at every stage what type of node we are dealing with. The compiler can use static typing to avoid common bugs from mixing up these types and fewer runtime sanity checks are needed. == Simplified Node Conversion == Converting nodes to operations was previously based on "relinking", i.e. nodes would start with by mirroring links in the Blender DNA node trees, then add operations and redirect these links to them. This was very hard to follow in many cases and required a lot of attention to avoid invalid states. Now there is a helper class called the NodeConverter, which is passed to nodes and implements a much simpler API for this process. Nodes can add operations and explicit connections as before, but defining "external" links to the inputs/outputs of the original node now uses mapping instead of directly modifying link data. Input data (node graph) and result (operations graph) are cleanly separated. == Removed Redundant Data Structures == A few redundant data structures have been removed, notably the SocketConnection. These are only needed temporarily during graph construction. For executing the compositor operations it is perfectly sufficient to store only the direct input link pointers. A common pointer indirection is avoided this way (which might also give a little performance improvement). == Avoid virtual recursive functions == Recursive virtual functions are evil. They are very hard to follow during debugging. At least in the parts this patch is concerned with these functions have been replaced by a non-virtual recursive core function (which might then call virtual non-recursive functions if needed). See for example NodeOperationBuilder::group_operations. | |||
2014-02-19 | Code cleanup: style | Campbell Barton | |
2014-02-13 | Fix T38529, Blur node size 0 doesn't work. | Lukas Tönne | |
The blur operations were clamping the filter size to 1, which prevents no-op blur nodes. Further any value < 1 would also be ignored and in many combinations the filter scale setting ("Size") would only work in integer steps. Now most blur settings will work with smooth Size value scaling as well, meaning you can choose a reasonably large filter size (e.g. 10) and then use the Size factor to scale the actual blur radius smoothly. Note that non-integer filter sizes also depend on the filter type selected in the Blur node, e.g. "Flat" filtering will still ignore smooth filter sizes. Gaussian filters work best for this purpose. | |||
2013-12-22 | Style Cleanup: remove preprocessor indentation (updated wiki style guide too) | Campbell Barton | |
2013-03-20 | Fix #34694. This was actually a bug in the compositor's Bokeh Blur ↵ | Lukas Toenne | |
operation. It was writing outside of allocated memory in case of (0, 0) size buffers, with the usual unpredictable results. | |||
2012-11-01 | style cleanup | Campbell Barton | |
2012-10-23 | use min_ max_ functions in more places. | Campbell Barton | |
also fix minor error in MOD decimate when the modifier did nothing the reported face count would be wrong. | |||
2012-08-16 | compositor: replace C++ new/delete with guardedalloc. | Campbell Barton | |
2012-08-10 | code cleanup: compositor - define size for executePixel function output ↵ | Campbell Barton | |
float array | |||
2012-08-04 | style cleanup | Campbell Barton | |
2012-08-03 | style cleanup: compositor | Campbell Barton | |
2012-07-19 | Fix for | Jeroen Bakker | |
* [#32040] size-input of a blur-node is uniform for the whole picture * [#32062] Blur node Size input is not working with * [#32140] Blur Node using a greyscale input as size multiplier fails to work Node now has a new option (new compositor cannot detect if the connected part is a single value, or an image connected). With this option the use of a reference image to multiply the size of the blur per pixel can be enabled/disabled. Regards, Jeroen - At Mind - | |||
2012-07-13 | Inner loop optimization of blur node | Jeroen Bakker | |
2012-07-13 | Removed parameter from executePixel and initializeTileData. | Jeroen Bakker | |
2012-07-12 | Small optimizations in compositor. | Jeroen Bakker | |
Most of them are not noticeable. | |||
2012-06-26 | use m_ prefix for compositor class members (all compositor operations). | Campbell Barton | |
2012-06-25 | fix for crash with blur - happened most when there was a size input, need to ↵ | Campbell Barton | |
mutex lock before allocating the gauss array. also add suspiciously missing call to BlurBaseOperation::initExecution, X had but Y was missing. | |||
2012-06-16 | code cleanup: spelling 'multiplyer' --> 'multiplier' | Campbell Barton | |
2012-06-16 | feather option for dilate/erode node - needed for alpha masks so we can ↵ | Campbell Barton | |
(blur in/out), currently only positive values supported. | |||
2012-06-15 | fix for other uninitialized values for the split viewer node as well as ↵ | Campbell Barton | |
incorrect frees for gaussian blue nodes. | |||
2012-06-15 | style cleanup: composite/blur | Campbell Barton | |
2012-06-13 | yse BLI_math for the compositor in more places. | Campbell Barton | |
2012-06-11 | * Blur node had some irregularities at the edge of the screen | Jeroen Bakker | |
only visible when doing large size blurs. also solved the catcom/mitch filter that didn't work at low/medium quality PS never use BokehBlur Gausian filter as it is 99%+ identicat as non bokeh <= top for sergey- :) | |||
2012-06-09 | code cleanup: reduce float/double promotion | Campbell Barton | |
2012-06-04 | Fix for relative blur size for non-fast gaussian blur methods. | Sergey Sharybin | |
Call base blur's initExecution from GaussianBokehBlurOperation and GaussianXBlurOperation | |||
2012-06-01 | Optimize Gaussian blurs | Monique Dewanchand | |
2012-05-18 | style cleanup: compositor, pointer syntax, function brace placement, line length | Campbell Barton | |
2012-05-17 | style cleanup: braces, compositor | Campbell Barton | |
2012-05-17 | ____ | Jeroen Bakker | |
`````|````` | | | ..'''' | | | |______ .'' | | | | ..' | | |_______ |___________ ....'' merge to TRUNK! * The old compositor is still available (Debug Menu: 200) This commit was brought to you by: Developers: * Monique Dewanchand * Jeroen Bakker * Dalai Felinto * Lukas Tönne Review: * Brecht van Lommel Testers: * Nate Wiebe * Wolfgang Faehnle * Carlo Andreacchio * Daniel Salazar * Artur Mag * Christian Krupa * Francesco Siddi * Dan McGrath * Bassam Kurdali But mostly by the community: Gold: Joshua Faulkner Michael Tiemann Francesco Paglia Blender Guru Blender Developers Fund Silver: Pablo Vazquez Joel Heethaar Amrein Olivier Ilias Karasavvidis Thomas Kumlehn Sebastian Koenig Hannu Hoffrén Benjamin Dansie Fred M'ule Michel Vilain Bradley Cathey Gianmichele Mariani Gottfried Hofmann Bjørnar Frøyse Valentijn Bruning Paul Holmes Clemens Rudolph Juris Graphix David Strebel Ronan Zeegers François Tarlier Felipe Andres Esquivel Reed Olaf Beckman Jesus Alberto Olmos Linares Kajimba Maria Figueiredo Alexandr Galperin Francesco Siddi Julio Iglesias Lopez Kjartan Tysdal Thomas Torfs Film Works Teruyuki Nakamura Roger Luethi Benoit Bolsee Stefan Abrahamsen Andreas Mattijat Xavier Bouchoux Blender 3D Graphics and Animation Henk Vostermans Daniel Blanco Delgado BlenderDay/2011 Bradley Cathey Matthieu Dupont de Dinechin Gianmichele Mariani Jérôme Scaillet Bronze (Ivo Grigull, Dylan Urquidi, Philippe Derungs, Phil Beauchamp, Bruce Parrott, Mathieu Quiblier, Daniel Martinez, Leandro Inocencio, Lluc Romaní Brasó, Jonathan Williamson, Michael Ehlen, Karlis Stigis, Dreamsteep, Martin Lindelöf, Filippo Saracino, Douwe van der Veen, Olli Äkräs, Bruno D'Arcangeli, Francisco Sedrez Warmling, Watchmike.ca, peter lener, Matteo Novellino, Martin Kirsch, Austars Schnore, KC Elliott, Massimiliano Puliero, Karl Stein, Wood Design Studios, Omer Khan, Jyrki Kanto, Michał Krupa, Lars Brubaker, Neil Richmond, Adam Kalisz, Robert Garlington, Ian Wilson, Carlo Andreacchio, Jeremias Boos, Robert Holcomb, Gabriel Zöller, Robert Cude, Natibel de Leon, Nathan Turnage, Nicolas Vergnes, Philipp Kleinhenz, Norman Hartig, Louis Kreusel, Christopher Taylor, Giovanni Remondini, Daniel Rentzsch, Nico Partipilo, Thomas Ventresco, Johannes Schwarz, Александр Коротеев, Brendon Harvey, Marcelo G. Malheiros, Marius Giurgi, Richard Burns, Perttu Iso-Metsälä, Steve Bazin, Radoslav Borisov, Yoshiyuki Shida, Julien Guigner, Andrew Hunter, Philipp Oeser, Daniel Thul, Thobias Johansson, Mauro Bonecchi, Georg Piorczynski, Sebastian Michailidis, L M Weedy, Gen X, Stefan Hinze, Nicolò Zubbini, Erik Pusch, Rob Scott, Florian Koch, Charles Razack, Adrian Baker, Oliver Villar Diz, David Revoy, Julio Iglesias Lopez, Coen Spoor, Carlos Folch, Joseph Christie, Victor Hernández García, David Mcsween, James Finnerty, Cory Kruckenberg, Giacomo Graziosi, Olivier Saraja, Lars Brubaker, Eric Hudson, Johannes Schwarz, David Elguea, Marcus Schulderinsky, Karel De Bruijn, Lucas van Wijngaarden, Stefano Ciarrocchi, Mehmet Eribol, Thomas Berglund, Zuofei Song, Dylan Urquidi ) |