didascalie-net / span

Spaces And Nodes - Environnement for real time media controls and organisation
didascalie-net.github.com/span
3 stars 0 forks source link

/span/load and autoload #37

Closed tmays closed 12 years ago

tmays commented 12 years ago

I have some questions/comments about the /span/load.

First, why is it 1/0 and not jut a bang? Usually actions are "bangs" unless it is an action that can be stopped, then a 1/0 would be appropriate. Unless you want to use "1" to be able to save it as an int in the db.

Other than that, I don't understand your "onebang" patch. what is it supposed to do? As it is, nothing happens with the first /span/load 1, nor with the first 0, then the next "1" tiggers, and also the next "0" triggers, and then 0 and 1 continue triggering each time. I don't understand what you want it to do and why?

And why does autoload check state and load the last project when /span/load is "0"?

It seems that you're using the /span/load as a command AND as a "done" indicator, but that is confusing. /span/load is a command, and we should have a /span/load/done when it is finished. I even think that the project loading should happen within the initial load sequence of /span/load. /span/load/done sould only happen after the project has been merged. Then tapemovie will start its loading sequence...

If you don't agree, could you explain?


In the load patch, we can remove the span.load argument of the dict. I don't remember if I put it there or you, but unless we plan on accessing it from another part of span, it's not necessary. If there's no argument it will just be given a uniue id, and since we only open it in the beginning to load span, that's all we need.