justingardner / mgl

A suite of mex/m files for displaying psychophysics stimuli
http://justingardner.net/mgl
Other
18 stars 22 forks source link

Implement new command model #93

Closed benjamin-heasly closed 7 months ago

benjamin-heasly commented 8 months ago

This introduces a new OOP command model to the Metal app, which I started documenting in a new readme. I'm hoping this makes for a nice separation of duties between the command interface, renderer, and specific commands.

This PR touches a lot of code! The raw diff might be hard to read. In addition to the new readme, here are some highlights that I hope will make reviewing the change easier:

While doing this refactor, I've had upcoming features in mind, like counter api timestamps and queued batches of commands. I haven't implemented these yet, but I think they will fit right in :-)

justingardner commented 8 months ago

That all sounds very sensible. Maybe you could walk me and Eli through it all next time we meet.

benjamin-heasly commented 7 months ago

I'm closing this for now but I'll open a new, similar PR soon.