Re: [Fieldtrip] Lighting Patch for Fieldtrip

Also, I'd also like to see some response to Ivan's last message on the thread "Adding lighting to FieldTrip". I appreciate Ivan's focus on GPU execution, which I hope will soon replace CPU execution in the implementation of FieldTrip. I expect we'll get a quite dramatic boost in performance as well as having more flexibility. - Conal On Tue, Jan 6, 2009 at 3:33 PM, Conal Elliott wrote:

Thanks, David! I'm glad for lights in FieldTrip. The code looks mostly good to me. A few comments:

* renderLightsIO's 'h' definition could use some prettying up: I'd prefer a unary style (e.g. 'h Empty = return') and using Kleisli composition ((>=>)) for the Union case.

* You've mostly aligned "=", ",", "::", and "$=", which I appreciate, as removing surface irregularities helps me see and fix deeper irregularities. You missed some, in renderLightsIO, pltToGL ("i"), LightType, and a few others. If you look carefully at the .patch, you'll find them all.

* I see lights getting enabled. How do they get disabled?

Would you mind addressing these issues and resending?

Happy new year,

- Conal

On Tue, Jan 6, 2009 at 2:28 PM, David Sankel wrote:

...

Hello Folks,

I've attached patch that will allow FieldTrip full access to omni, spot, and directional lights with varying parameters. It works much the same way as geometry does. See examples.hs for neat examples (the LegacyAdapterAnim.hs file is required for the examples and is different from the one I posted earlier).

I'll be updating the ticket (http://trac.haskell.org/FieldTrip/ticket/16) shortly.

David

-- David Sankel

_______________________________________________ FieldTrip mailing list FieldTrip@haskell.org http://www.haskell.org/mailman/listinfo/fieldtrip