(VisualRenderer) Forcibly ignore shared UBO if we call RegisterVisualTransformUniform()

Since the change of Pipeline give overhead for real world applications,
let we just forcibly ignore shared UBO if we don't use default visual UBO cases.

Change-Id: I1edec4752c54d55f91b61f6a12c8ccebf860e32f
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Print logs when scene initialize and uninitialized + print log if render target is nullptr

Change-Id: I61f651aa70431fb0f0095609077ec92a4ef52698
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

DALi Version 2.4.17

Change-Id: I13f6ace58b099267022c503b00a4dd4e728baaf4

Merge "Removed unnecessary fomat file as it breaks some clang builds" into devel/master

Removed unnecessary fomat file as it breaks some clang builds

Change-Id: I40467afda7d07731947b9c456560444af2da8686

Merge "Reset uniform blocks flags at post render time" into devel/master

AlphaFunction for spring animation

Change-Id: Ie25f268977c6b5e87ad57e5cae2279adbbba36c5
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

Reset uniform blocks flags at post render time

Change-Id: Ia16f4eeb8f5eb6c28fbd172a9eb20a47aaa9ad37
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "The tap gesture is not continuous. Since it is a gesture that occurs when up, CancelProcessing is not needed." into devel/master

Merge "Change OffscreenRenderableType change by Register, instead of Set" into devel/master

The tap gesture is not continuous. Since it is a gesture that occurs when up, CancelProcessing is not needed.

Change-Id: I6b16405f5b884999dabc5fc430fb93902bec8680

Change OffscreenRenderableType change by Register, instead of Set

We have multiple objects to try to change the custom-actor's OffscreenRenderableType.
Let we count the number of type called by bitfield, and check the enum by the counts.

Change-Id: I95049c17007a878a21316df73476cbe5841a1138
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Calculate align size at UniformBuffer, not external

We don't need to give alignment ownership of memory offset to outsize.

Let we open the size freely, and calculate offset internally.

Change-Id: I7cd0ef88f15e716a75b7caff74830e35e3da076f
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Make multiple renderer share the list of uboView + Recycle uboView" into devel/master

DALi Version 2.4.16

Change-Id: I837190b868dfdcd71c8577c4bf1025ed489eba42

Make multiple renderer share the list of uboView + Recycle uboView

Let we reduce the allocation of ubo view list

Also, make we recycle same pointer of temperal uboView,
so we can reduce reallocate memory from pool.

Change-Id: Idb28c6227350a36ebc7c4e537fd90e385bb903f9
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Added FILE_CACHE_SUPPORT hint for shader" into devel/master

Merge "Make new range for AnimatableProperty registration without uniform mapping" into devel/master

Added FILE_CACHE_SUPPORT hint for shader

Previously, all custom Shaders supported file caching, which was not the intended behavior.
therefore, we added a flag to distinguish between internally generated code and externally generated code.
if you want to enable File Caching for externally generated shader code, you can do so using this hint.
(however, the validity of the shader must be verified by the app itself.)

Change-Id: I5e0cb0c6f4177da8159c50e2e27a0833b2c6e54f

Ensure to call program.GetUniform() only 1 times even if we found failed.

Until now, UniformIndexMap state mark as initialized = false if given unform
not found at shader.

In this case, we always try to re-search about given uniform name, which is useless.

To avoid this case, let we make the uniform found result as enum,
and so we can ensure that program.GetUniform only 1 time.

Change-Id: I23a180980f556e3f05302622453258f87742344d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Make new range for AnimatableProperty registration without uniform mapping

Change-Id: I8039aceafea494e4eb514915bd05a7aaea77a35d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "DALi Version 2.4.15" into devel/master

DALi Version 2.4.15

Change-Id: Idec53ed89316aa8ac9af6e55b6da1909ac63fca3

Add RenderInputCallback worldColor value

Let we include world color infomation of actor,
so native draw callback use the color infomation

Change-Id: Iccd1057e35fcbc925091fefbeee50ea65bb57ba0
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fix typo error (2)

Sorry :(

Change-Id: I88c1b22aef6c58a0f6617617699f3197c3eeea02
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fix typo error

Change-Id: I262c32601df7eb2c993f0ec54bb3171d082404da
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Make UBOV2::AlignSize() API as inline

Since we don't change mAlighment ever, we can mark it as const,
and move calculate Aligned size as inline function.

Change-Id: I7aeab753325aff16dca19ab53d13e9cb9f94d975
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Use UniformBufferView memory pool" into devel/master

Merge "DALi Version 2.4.14" into devel/master

DALi Version 2.4.14

Change-Id: I0e18651e8ae0d091c711b3b9fba8bf9fd67aa8f7

Use UniformBufferView memory pool

Per each Renderer::Render time, we create UniformBufferView the number of uniform blocks,
and destroy at that time.

Since we create & destroy UboView very frequency every frame, we'd better
keep this memory as pool system.

Moreover, let we remove some useless values and operations relative with
UBO View.

Let we make UboView.Write's offset parameter is relative of
View itself's offset, instead of UboBuffer's offset.

It will reduce useless offset getter calling

Change-Id: I583728ec1bd4cf0a1a34f6002a647f4f15a32e12
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Destroy renderer's node index map if node detached

It will reduce the time of Renderer with single, or multiple Node connected case.

It was useless iterate when renderer with single node (==usual cases) connected
and detached.

Instead, let we just destroy whole container, and re-generate at next render loop.

Change-Id: Ied6dd91b97ce4f693e5a004acfdec11b59bc3f80
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Make HashUtil implements as cpp, So reduce duplicated implmenets

Let we make basement of hash calculate optimization

+

Include below patchset, due to line coverage
https://review.tizen.org/gerrit/c/platform/core/uifw/dali-core/+/322289

Change-Id: I8b4679b737f5adfa0c09932af17f02d61bdea8a4
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Merge "Use swap for some cases instead move assign." into devel/master

Use swap for some cases instead move assign.

Since library don't make sure that moved std::vector container is cleared.

To make ensure, let we use std::vector::swap() instead move assign with clear.

Change-Id: I88742bc6cea8f32f15db6e733278f103467792e6
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Print some critical warning message as error to print at release mode

Since DALI_LOG_WARNING didn't print any logs at release mode,
several critical errors didn't print any infomations.

To make ensure those are real error, let we change the
log level as 'ERROR' and allow to print at release mode binary.

Change-Id: Ib74c21979799a53d4288d87feb849360fd3f9072
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Fix svace - Assert if RenderItem.mNode is nullptr + Print log if sceneobject is nullptr" into devel/master

Merge "GeometryTouchEvent :InterceptTouchEvents are propagated in a parent-child relationship." into devel/master

GeometryTouchEvent :InterceptTouchEvents are propagated in a parent-child relationship.

Change-Id: Iffe7d8d5aca362ce9fee7e0fed4eac28d87c15cf

Fix svace - Assert if RenderItem.mNode is nullptr + Print log if sceneobject is nullptr

Change-Id: Ib85427f2605a6685e4a0c37142195ab82361c399
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Give range of tags for constraints" into devel/master

Merge "Adding UniformBlock for shared uniforms" into devel/master

Add Rendered flag for PreRender the Scene

Let we make some interface whether given scene rendered previous frame or not.

Until now, we only had "current" scene has render instruction to scene or not.
But also, we should clear the scene if "previous" scene had render instruction
but not now.

Since it could not be known by single boolean value, we make another class
to communicate the PreRender result.

By this infomation, we should determine whether we should call
ClearScene() or not even if fullSwap keyword not applied.

Furthermore, Let we make PreRender(Scene) API name as PreRenderScene,
pairwise with RenderScene().

Change-Id: Iab820942aff1d136cef1ab111901dc99c7ccdcf7
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "(Animation) Allow to print debugging logs for release mode" into devel/master

Adding UniformBlock for shared uniforms

Let we make UniformBlock class and connect it to Shader.

The UniformBlock who have same name with at shader, will ignore
the value of Actor - Renderer - Shader propert chain. And will use
UniformBlock itself's value.

All renderer who has same shader with connected uniform block, will have
same uniform values at rendering time.

It will reduce the number of UniformWrite call at render thread.

Note : Since Uniform have dependency with Program, we should clear
cached Program at program-controller whenerver we try to connect uniform blocks
to shader.

TODO : We should notify to Graphics::Program s.t. given uniform block is shared.

The binding value of shared uniform block
 - glUniformBlockBinding(~~, ~~, binding);

should not call BindBufferRange in future, match as

 - glBindBufferRange(~~, binding, ~~~~);

Cherry-pick of patchset : https://archive.tizen.org/gerrit/c/platform/core/uifw/dali-core/+/294788

Change-Id: I9b0dab139a7064bb8889db9693e4f84ad74185ad

Give range of tags for constraints

Let we give constraint tag's range for custom and internal.

The tags what we can control as public is only for custom ranges,
and make internal constraint didn't removed.

Change-Id: I0ab45d4ed258cdcd30f4c98a61b0a0251cf4c692
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.4.13

Change-Id: I7d7acdc14f56d7866bc3d4274e8d6c27caafa7a2

(Animation) Allow to print debugging logs for release mode

Let we print animation relative logs if
- trace enabled
- DALI_TRACE_PERFORMANCE_MARKER enabled
- DALI_LOG_DEBUG_INFO printable

Change-Id: I1619b265bda50e6a98a45eddbb14b954ee023237
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "(dali_env) Check if vulkan is installed when outputting setenv" into devel/master

Merge "Hit Test for OffScreenRendering" into devel/master

Hit Test for OffScreenRendering

Change-Id: I32c7c00def338a804cf118436a2de13b11b8f4cc
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

(UnsafeDirectRendering) Reset geometry cache after direct native draw

Change-Id: If8d398b12a0ea477794defff703c6b81f4e86cd5
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

(dali_env) Check if vulkan is installed when outputting setenv

Change-Id: I374cc14ffe3dd09d313411a70ca7e10a18e78313

Remove error message when we set/get AnimatableProperty

We only need to call setFunc/getFunc only if it exist.
Until now, it check as error if sefFunc/getFunc not exist.

Change-Id: I534e9dd2f7b76e9a50db1935931eea99cc1b70ca
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Merge "DALi Version 2.4.12" into devel/master

DALi Version 2.4.12

Change-Id: I188963c11f5fbc4624282a535801da9238696313

Allow animatable property getter and setter functions

Change-Id: I2b314566bb1ecfffad9533bee61d085289604788
Signed-off-by: jmm <j0064423.lee@samsung.com>

(Constraint) Bake constraint property only if value changed

Change-Id: I8aa0bd9b0a5f2f41f0466f8bba278ac978327946
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Set RendererAdded value at UpdateStatus

Change-Id: Ia6dd039946ed952585beb8be54b908201acde0de
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Ensure to SceneObject initialized

Change-Id: Ia8a6a842840def7a758adaf45f0957e0a6a171de
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "(HitTest) Use l-value or raw pointer of Internal::Actor, instead IntrusivePtr" into devel/master

Merge changes I0c29d9e4,Iee4f76dc,Id6ea7306 into devel/master

* changes:
  Core::PreRender returns true if something is renderable to scene
  Adding clear color to scene-graph Scene
  Debugging render scene

(HitTest) Use l-value or raw pointer of Internal::Actor, instead IntrusivePtr

Since std::atomic is expensive, we need to touch him less times if we can.

Hit test logic only works internally. So we could remove IntrusivePtr usage.

The only problem is HitTestResult callback case.
But for now, it used less usage, so we can ignore this cases.

Change-Id: Iaff07402401ecde339629068f52406dd1105a00d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Invert Y when the camera renders on Framebuffer and y invert is true.

 - When scene is drawn on window (with viewport), DALi draws scene on window directly so picking ray and result is always fit.
 - When scene is drawn on FBO, DALi cannot know how the result is drawn by app, so picking ray is always on space that has (0, -1, 0) as UP.

Change-Id: I441356c5b128711ca864a9cd2d51e166f297d245
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

Merge "Hit Test Fix for layered Hit test" into devel/master

Core::PreRender returns true if something is renderable to scene

Change-Id: I0c29d9e4ed1034b622e130d347a157f1e3f6633b

Merge "(DirectRendering) Send rotated clippingBox so we can call glScissor without convert" into devel/master

Adding clear color to scene-graph Scene

Change-Id: Iee4f76dcb41feab4d0bce7d8bc2bc4cbbe7bcc58

Debugging render scene

Added more debug to RenderScene under LOG_RENDER_MANAGER tag

Change-Id: Id6ea7306dc166a438abe8033b7015e1f057a16e8

Merge "DALi Version 2.4.11" into devel/master

DALi Version 2.4.11

Change-Id: Ia6e5fabdfc964527bc5517822530c07345eb4136

(DirectRendering) Send rotated clippingBox so we can call glScissor without convert

Let we make RenderCallbackInput's clippingBox is screen coordinates.

It will make that we don't consider window surface rotated or not.

Change-Id: I9d7e0c89682a4de12fad6bb57d322b3eaafb5d3c
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Support UpdateAreaExtents for Renderer

Add property for extents the dirty rect calculated range for the renderer.

Change-Id: Ibdfe6ea2a90906d01d29c93b0e632d5e631249d4
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Hit Test Fix for layered Hit test

Change-Id: Id2611e20a20259dcfbd341fa37b3029cd326e3cc
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

(Animation) Calculate AutoReverse looping mode progress at Animation time, not Animator.

Until now, we control the looping mode's progress at Animator side.
It make some mis-implements when we add newly animators.

AnimateTo(A)
SetLoopingMode(AUTO_REVERSE) ///< A become auto-reversed..
AnimateTo(B)                 ///< B is not auto-reversed!!

Since LoopingMode is Animation's property, we should apply newly added animators.

But actually, Animation know all informations, so I think Animation could control
the calculation logics.

Change-Id: I5820985f59066a279a59bd9c19adfede068fde59
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "DALi Version 2.4.10" into devel/master

DALi Version 2.4.10

Change-Id: Ibccc817080227fd72e93a944ebe2e49a87e2cdaf

When hoverevent of geometry type is used, DispatchHoverMotion is not handled properly when false. This is fixed.

Change-Id: Ibcd68776f55a6779485a36d2d17b030801225b7f

If IsDispatchHoverMotion is false, the move event was not dispatched. So consumedActor should keep the previous LastConsumedActor value.

Change-Id: Ifbadde1b21abfca728c9e91be81a8f57c4650a6e

DALi Version 2.4.9

Change-Id: I33fb7c331cf4aab53d6a787060cc409d12ef99f9

Allow to create renderer without geometry and shader

Let we make constructor renderer without geometry and shader.

Also, check some codes s.t. we can allow nullptr mGeometry and mShader.

Change-Id: I99eb3011c4cb7f1a3c9ff69f90842c44005048f6
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Clear pipelie cache if geometry destroyed or buffer changed

We also cache pipeline cache the Render::Geometry by the raw pointer.
If someone use duplicated pointer, it might return invalid pipeline
with unmatched vertexInputState.

To avoid this issue, let we erase cache if Render::Geometry destroyed,
same as Render::Program

 - This is the commit message #2:

Reset cached pipeline if geometry buffer changed

Until now, we don't re-cache the geometry
if the vertex buffer added/removed, or data changed.

Since the vertex attribute might be changed if we try to use
same geometry, the rendering result
show some non-common results.

To fix this cache issue, let we ensure to reset the cached infomations
if the vertex buffer / indices buffer changed.

Change-Id: I0dc5b4fb6b0645d4b7763d7aa890d6ad946d54c6
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.4.8

Change-Id: I42df6d888f5101778ba29ad50cafed19e33e9746

Merge "Make PixelData Stride return byte size, instead of pixel" into devel/master

Increase warning message threshold for animator

When we launch benchmark.example, it print too much warning logs :(

Change-Id: I95a8558710ea219fe478d92c567e12f415536373
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Make PixelData Stride return byte size, instead of pixel

Some cases doesn't have stride byte as multiply of BytesPerPixel().
But until now, PixelBuffer and PixelData only support Stride as pixels count
(It looks since GL_UNPACK_ROW_LENGTH is pixel scale, not a bytes.)

For example, If we have 2 width image with RGB888 format, stride byte
should be 8 byte, which is not a multiply of 3 byte.

To support those cases, let we set stride value as bytes scale.

Change-Id: I27aa9757e7320475b7f0519a62262d1ae2f7e07b
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Merge "DALi Version 2.4.7" into devel/master

DALi Version 2.4.7

Change-Id: Id6252a283cf17a78688548f01d0263369e292bcc

Merge "(Partial Update) Ensure to renderer dirty if Geometry changed" into devel/master

Merge "Reduce IntrusivePtr<ActorPtr> reference change during PropagateAll" into devel/master

Print more debug logs for Animation

Let we print more infomations for dali animations
whenever we call API

Change-Id: I5c7db1144adff6b742536ae08a760f066a7cac3e
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

(Animation) Bug fix when Cleared state animation finish normally don't emit signal

Their was some bug if

(Event Thread) Clear()
(Render Thread) Finish animation normally
(Event Thread) NotificationManager->ProcessMessages() -> HasFinished() called
...
(Event Thread) Play()
(Render Thread) Finish animation normally
(Event Thread) NotificationManager->ProcessMessages() -> HasFinished() called

progress comes.

The first HasFinished() should not emit finished signal.
But second HasFinisehd() should emit finished signal, but not.

It was bugs since HasFinished() update some values, but we don't use it.
(Cleared state should have mNotificationCount always) It make bug.

Change-Id: I593c9983fe97924638c85f0ef30d3c98064d9415
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Reduce float hash collision by squaring valueHash

Applied valueHash^2 to reduce frequent hash collisions in similar float values.

Change-Id: I8877b24e2936e8a3cbf7da3ca52998f74d75e65c

Merge "Share partial update data info to RenderItem" into devel/master

DALi Version 2.4.6

Change-Id: I21a91c30091c3b411e9068c450921be029fd4cb1

(Partial Update) Ensure to renderer dirty if Geometry changed

Let we also make item.mRenderer is Updated() if Render::Geometry information
changed.

For example, Toolkit::TextEditor change the vertex buffer data and indicise
when some texts, or highlighted area are changed.
But if the size of each actors are not be changed, the infomation of
geometry changed not be applied to the PreRender() dirtyrect calculation.

To fix this issue, let we make ensure that geometry is updated at Upload time,
and notify it to Render::Renderer's Update() time.

Change-Id: I67ff3c9e7a155703fba6b3be8b80d5aede8ce02d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Share partial update data info to RenderItem

Let we make RenderItem use Node's PartialUpdateData directly,
instead copy the matrix and etc.

It will reduce each item's memory, and memory copy time.

Change-Id: I42e12b24a6429734f7754db5880524b8bc4e4f32
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Reduce IntrusivePtr<ActorPtr> reference change during PropagateAll

std::atomic value changeness might gave overhead if we call it
very frequencly.

To reduce it's overhead, let we use Interal::Actor& instead Actor handle
if we can.

Note that we cannot remove Dali::Actor handle at RelayoutInfo,
since we need to guard case that Actor itself destruct during
Relayout callback at application side.

Change-Id: I5f058be4706c8c2be77f1677e1f5624f37d50478
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Reduce dlog spam per frame

Added debug filter for  animation update and transform manager.
These can be turned on by setting DALI_LOG_ANIMATION or
DALI_LOG_TRANSFORM_MANAGER environment variables to 3 before running.

(Or temporarily turned on in a build by modifying the filter
constructor arguments).

Change-Id: I1ade7dfecb5fd0b9d4a7dc8d043486af52a791bf

Check if scene has something to render

Core::PreRender(scene) now returns true if there is any render instruction
with a render list with render items for that scene.

Change-Id: Ib628f0a03dc9959fc945bdb410a7af52ec54e86b