On Apr 2, 2012, at 6:43 PM, crdueck wrote:

> Hello all, I've posted a complete first draft of my GSoC application. 
> It's mostly finished in terms of content for now, but I'd love to get 
> some feedback on anything that can be improved or added.

That's an excellent level of detail with answers to just about everything we've 
suggested.  Thanks for sharing it, Chris.  I'd suggest submitting it to 
google-melange if you haven't yet, even if it's still in draft form, just to 
officially have it "in the system" before the deadline.  Include a statement 
near the top saying that it's a draft if you like, but also include a link to 
the patch you posted.

We did find one technical issue with the patch where unrelated/unintentional 
edits got included.  More feedback on the patches tracker.

As for the proposal content -- I'm curious why you chose to implement both 
volume and centroids for just some of the primitives instead of just one or the 
other for all of them.  What's the reasoning there?  I'd also like to see a 
little more detail on how you're going to test your implementations for 
correctness and how it'll be exposed to a user.  The mged/archer "analyze" 
command may be a natural starting place (src/libged/analyze.c).

Cheers!
Sean


------------------------------------------------------------------------------
Better than sec? Nothing is better than sec when it comes to
monitoring Big Data applications. Try Boundary one-second 
resolution app monitoring today. Free.
http://p.sf.net/sfu/Boundary-dev2dev
_______________________________________________
BRL-CAD Developer mailing list
brlcad-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/brlcad-devel

Reply via email to