search

ISET / iset3d-v3

LEGACY: Replaced by iset3d (v4) -- Read a PBRT file. Edit the parameters. Render an ISET scene or optical image.
MIT License
36 stars 8 forks source link

s_ versus t_ #24

Closed DavidBrainard closed 5 years ago

DavidBrainard commented 5 years ago

I continue to be baffled by what conceptual distinction scrips are sorted into s and t scripts.

wandell commented 5 years ago

The scripts were supposed to be interesting examples of calculations that we might want to share with others. The tutorials were supposed to be real training cases with explanatory text. As your comment makes clear, there is no bright line between scripts and tutorials. A beautifully commented script could be a tutorial. A lousy tutorial should be a script.

Not sure what we should do about this. I continue to think, however, that we need tutorials. Should we create ISETBio-scripts and only move something into ISETBio when it is a tutorial? In a way, the scripts directory serves that purpose already. Continuing the bafflement ....

DavidBrainard commented 5 years ago

Thanks. I am beginning to understand the concept. Agree that it is tricky to know exactly how to divide the continuum. Something to discuss, maybe with Jenn as well.

We experimented in IBIOColorDetect with making tutorials callable functions that could do actual work with the right arguments, but that had defaults that did something simple and illustrative. But I think maybe in then end that made them too hard as tutorials, as they started to get pretty complex. At the same time, making them functions seemed like a step in the direction of minimizing code duplication.