2015-05-03 15:42 GMT+02:00 Vasco Alexandre da Silva Costa <vasco.co...@gmail.com>: > This is just my opinion but I think you should have primitive specific > methods where possible as they will be much more computationally efficient.
This is the restriction: "where possible". It isn't possible for every type, e.g. combination. Therefore we need a general method. On the other hand such a method would be a good generic rt_functab entry for primitives were we haven't a specialized function yet. > On Sun, May 3, 2015 at 10:15 AM, Kalpit Thakkar <ceasy...@gmail.com> wrote: >> >> Hello Daniel! >> I wanted to discuss about the approach that could be followed in enhancing >> the Geometry Engine by adding the support for Volume, Centroid and Surface >> Area callbacks for primitives. >> As we had talked before, it could be a general method or it can be >> implementing the missing functions in librt. In any case, combinations >> *MUST* be entailed and hence general method seems to be the better choice. Implementing the missing functions includes to implement a generic one too. However, you could start with implementing primitive specific functions if you know how (i.e. it is relatively easy for you) to gain experience in doing so. This could be a good patch to demonstrate your programming skills to get commit access. >> However, I would like to discuss the pros and cons of the general method >> and about where to look so that I can understand it nicely. >> Please suggest a time when we can meet up on IRC and discuss the same. >> >> About the interaction schedule : I think it'll be great if we can meet up >> on IRC atleast twice a week when I'll update you with the progress and >> discuss about the difficulties faced. I suggest Monday and Friday. Ofcourse, >> you can always check the development log (http://brlcaddev.wordpress.com) >> whenever required. At least in May and June I'm traveling a lot, so I cannot promise to be regular available on IRC. But e-mail via the mailing list, like this one, is fine. And there are other competent mentors too. >> About the commit access : I promised you that I would merge the Pipe >> implementation done by Andrei Popescu. I have applied the indentation rules >> on the test files and the final patch is ready. I'll upload it and after >> reviewing, if it's ready for merging, it'll be a delight. So, please tell me >> what to do about it. If it's OK I'll commit it. (I want to get it done too.) >> Last but not the least : I feel privileged to have this opportunity. I'm >> glad I now have a real chance to make a difference. Muchas Gracias :D You are welcome :) Regards, Daniel ------------------------------------------------------------------------------ One dashboard for servers and applications across Physical-Virtual-Cloud Widest out-of-the-box monitoring support with 50+ applications Performance metrics, stats and reports that give you Actionable Insights Deep dive visibility with transaction tracing using APM Insight. http://ad.doubleclick.net/ddm/clk/290420510;117567292;y _______________________________________________ BRL-CAD Developer mailing list brlcad-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/brlcad-devel
