<div dir="ltr">Hi Julien,<br><div class="gmail_extra"><br></div><div class="gmail_extra"><div>I have begun my old build tests and have merged master into osganimation branch and checked this in, and then fixed the build problems when compiling with OSG_USE_REF_PTR_IMPLICIT_OUTPUT_CONVERSION set to OFF.  These changes are all checked into the osganimation branch.</div><div><br></div><div>I have begun looking at source code and find the style used a bit awkward and rather different from the rest of the OSG.  I know this will have been partly been down to what it's been like prior to your changes.  I don't find the style used for spacing and indentation helps readability - there are some very odd choices where therearenowspacesbetweenstuff and then      bits     that have     an        odd     segments of     code       where the spacing     is        very      spacious for     no apparent      reason.</div><div><br></div><div>I don't know how practical it is for me to go through all osg osgAnimation and headers to fix things.  In the past I have used code beutifiers with varying success.  Perhaps there are modern tools now that can do a better job and worth trying out.  I welcome suggestions on what tools there might be.  What I'd like to arrive at is an osgAnimation headers and source that look like the rest of the OSG.<br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 27 October 2017 at 14:27, Julien Valentin <span dir="ltr"><<a href="mailto:julienvalentin51@gmail.com" target="_blank">julienvalentin51@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Robert<br>
I just did a commit of my experimental useless bone removal method.<br></blockquote><div><br></div><div>There is a conflict on this change that github is reporting:</div><div><br></div><div>   <a href="https://github.com/openscenegraph/OpenSceneGraph/pull/374">https://github.com/openscenegraph/OpenSceneGraph/pull/374</a></div><div><br></div>Thoughts?<br></div><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
So long so far I didn't have any issue with my Blender test models (which represent a "outnumbered sample" of 2 models...one with ik the other with morphtarget)<br>
However, I didn't have any feedback from the community or original maintainer so I can't certificate it 100% bugfree.<br>
<br>
>From my point of view, I achieve to do whadIwannado with it (MDI crowd) so I'm quite satisfied with the current version and would vote for merging, but it's just one perspective among other ...<br></blockquote><div><br></div><div>OK.  So we are now at a point where we need wider testing.<br></div><div><br></div><div>Personally I have not been a serious user of osgAnimation so haven't got much experience or test cases to draw from so need yourself and the rest of the community to hammer it hard.</div><div><br></div><div>Robert.<br></div><div> </div></div><br></div></div>