 trunk/TODO 2011/05/05 08:18:26 1120
+++ trunk/TODO 2011/05/10 13:32:53 1167
@@ 1,73 +1,67 @@
NOTE: GLK's approximate ranking of 5 most important tagged with
+NOTE: GLK's approximate ranking of 8 most important tagged with
[GLK:1], [GLK:2], ...
==============================
other SHORT TERM ============= (including needed for LIC)
==============================

Add a clamp function, which takes three arguments; either three scalars:
 clamp(x, minval, maxval) = max(minval, min(maxval, x))
or three vectors of the same size:
 clamp([x,y], minvec, maxvec) = [max(minvec[0], min(maxvec[0], x)),
 max(minvec[1], min(maxvec[1], y))]
This would be useful in many current Diderot programs.
One question: clamp(x, minval, maxval) is the argument order
used in OpenCL and other places, but clamp(minval, maxval, x)
would be more consistent with lerp(minout, maxout, x).

Level of differentiability in field type should be statement about how
much differentiation the program *needs*, rather than what the kernel
*provides*. The needed differentiability can be less than or equal to
the provided differentiability.
+========================
+SHORT TERM ============= (*needed* for streamlines & tractography)
+========================
[GLK:1] Add sequence types (needed for evals & evecs)
+[GLK:3] Add sequence types (needed for evals & evecs)
syntax
types: ty '{' INT '}'
value construction: '{' e1 ',' … ',' en '}'
indexing: e '{' e '}'
IL support for higherorder tensor values (matrices, etc).
 tensor construction [DONE]
 tensor indexing [DONE]
 tensor slicing
 verify that hessians work correctly [DONE]
+[GLK:4] evals & evecs for symmetric tensor[2,2] and
+tensor[3,3] (requires sequences)
Use ∇⊗ etc. syntax
 syntax [DONE]
 typechecking
 IL and codegen
+ability to emit/track/record variables into dynamically resized
+runtime buffer
test/uninit.diderot:
documents need for better compiler error messages when output variables
are not initialized; the current messages are very cryptic
+tensor fields: convolution on general tensor images
determinant ("det") for tensor[3,3]
+========================
+SHORTISH TERM ========= (to make using Diderot less annoying to
+======================== program in, and slow to execute)
expand trace in mid to low translation
+valuenumbering optimization [DONE, but needs more testing]
valuenumbering optimization
+Allow ".ddro" file extensions in addition to ".diderot"
Add type aliases for color types
 rgb = real{3}
 rgba = real{4}
+Be able to output values of type tensor[2,2] and tensor[3,3]
+(currently only scalars & vectors)
==============================
MEDIUM TERM ================== (including needed for streamlines & tractography)
==============================
+[GLK:1] Add a clamp function, which takes three arguments; either
+three scalars:
+ clamp(lo, hi, x) = max(lo, min(hi, x))
+or three vectors of the same size:
+ clamp(lo, hi, [x,y]) = [max(lo[0], min(hi[0], x)),
+ max(lo[1], min(hi[1], y))]
+This would be useful in many current Diderot programs.
+One question: clamp(x, lo, hi) is the argument order used in OpenCL
+and other places, but clamp(lo, hi, x) is much more consistent with
+lerp(lo, hi, x), hence GLK's preference
+
+[GLK:2] Proper handling of stabilize method
[GLK:1] evals & evecs for symmetric tensor[3,3] (requires sequences)
+allow "*" to represent "modulate": percomponent multiplication of
+vectors, and vectors only (not tensors of order 2 or higher). Once
+sequences are implemented this should be removed: the operation is not
+invariant WRT basis so it is not a legit vector computation.
[GLK:2] Save Diderot output to nrrd, instead of "mip.txt"
+implicit type promotion of integers to reals where reals are
+required (e.g. not exponentiation "^")
+
+[GLK:5] Save Diderot output to nrrd, instead of "mip.txt"
For grid of strands, save to similarlyshaped array
For list of strands, save to long 1D (or 2D for nonscalar output) list
For ragged things (like tractography output), will need to save both
complete list of values, as well as list of start indices and lengths
to index into complete list
[GLK:3] Use of Teem's "hest" commandline parser for getting
+[GLK:6] Use of Teem's "hest" commandline parser for getting
any input variables that are not defined in the source file
[GLK:4] ability to declare a field so that probe positions are
+[GLK:7] ability to declare a field so that probe positions are
*always* "inside"; with various ways of mapping the known image values
to nonexistant index locations. One possible syntax emphasizes that
there is a index mapping function that logically precedes convolution:
@@ 76,48 +70,24 @@
F = bspln3 ⊛ (img ◦ mirror)
where "◦" or "∘" is used to indicate function composition
extend norm (exp) to all tensor types [DONE for vectors and matrices]

ability to emit/track/record variables into dynamically resized
runtime buffer

Want: allow X *= Y, X /= Y, X += Y, X = Y to mean what they do in C,
provided that X*Y, X/Y, X+Y, XY are already supported.
Nearly every Diderot program would be simplified by this.
+Level of differentiability in field type should be statement about how
+much differentiation the program *needs*, rather than what the kernel
+*provides*. The needed differentiability can be less than or equal to
+the provided differentiability.
Want: nontrivial field expressions & functions:
 image(2)[2] Vimg = load(...);
 field#0(2)[] Vlen = Vimg ⊛ bspln3;
to get a scalar field of vector length, or
 field#2(2)[] F = Fimg ⊛ bspln3;
 field#0(2)[] Gmag = ∇F;
to get a scalar field of gradient magnitude, or
 field#2(2)[] F = Fimg ⊛ bspln3;
 field#0(2)[] Gmsq = ∇F•∇F;
to get a scalar field of squared gradient magnitude, which is simpler
to differentiate. However, there is value in having these, even if
the differentiation of them is not supported (hence the indication
of "field#0" for these above)

Want: ability to apply "normalize" to a field itself, e.g.
 field#0(2)[2] V = normalize(Vimg ⊛ ctmr);
so that V(x) = normalize((Vimg ⊛ ctmr)(x)).
Having this would simplify expression of standard LIC method, and
would also help express other vector field expressions that arise
in vector field feature exraction.
+Use ∇⊗ etc. syntax
+ syntax [DONE]
+ typechecking
+ IL and codegen
tensor fields: convolution on general tensor images
+Add type aliases for color types
+ rgb = real{3}
+ rgba = real{4}
==============================
other MEDIUM TERM ============ (needed for particles)
+MEDIUM TERM ================== (*needed* for particles)
==============================
[GLK:5] runtime death of strands; test/iso2d.diderot provides
great initial test for "die" command.

Put small 1D and 2D fields, reconstructed by tent, into
faster texture buffers

runtime birth of strands
"initially" supports lists
@@ 125,36 +95,59 @@
"initially" supports lists of positions output from
different initalization Diderot program
spatial data structure that permits strands' queries of neighbors
+Communication between strands: they have to be able to learn each
+other's state (at the previous iteration). Early version of this can
+have the network of neighbors be completely static (for running one
+strand/pixel image computations). Later version with strands moving
+through the domain will require some spatial data structure to
+optimize discovery of neighbors.
+
+============================
+MEDIUMISH TERM ============ (to make Diderot more useful/effective)
+============================
proper handling of stabilize method
+Python/ctypes interface to runtime
test/vrkcomp2.diderot: Add support for code like
+support for Python interop and GUI
 (F1 if x else F2)@pos
+Allow integer exponentiation ("^2") to apply to square matrices,
+to represent repeated matrix multiplication
This will require duplication of the continuation of the conditional
(but we should only duplicate over the liverange of the result of the
conditional.
+Alow X *= Y, X /= Y, X += Y, X = Y to mean what they do in C,
+provided that X*Y, X/Y, X+Y, XY are already supported.
+Nearly every Diderot program would be simplified by this.
+
+Put small 1D and 2D fields, when reconstructed specifically by tent
+and when differentiation is not needed, into faster texture buffers.
+test/illustvr.diderot is good example of program that uses multiple
+such 1D fields basically as lookuptablebased function evaluation
+
+expand trace in mid to low translation
+
+extend norm (exp) to all tensor types [DONE for vectors and matrices]
+
+determinant ("det") for tensor[3,3]
add ":" for tensor dot product (contracts out two indices
instead of one like •), valid for all pairs of tensors with
at least two indices
==============================
other MEDIUM TERM ============
==============================
+test/uninit.diderot:
+documents need for better compiler error messages when output variables
+are not initialized; the current messages are very cryptic
want: warnings when "D" (reserved for differentiation) is declared as
a variable name (get confusing error messages now)
support for Python interop and GUI

Python/ctypes interface to runtime

==============================
LONG TERM ====================
==============================
+LONG TERM ==================== (make Diderot more interesting/attractive from
+============================== a research standpoint)
+
+IL support for higherorder tensor values (matrices, etc).
+ tensor construction [DONE]
+ tensor indexing [DONE]
+ tensor slicing
+ verify that hessians work correctly [DONE]
Better handling of variables that determines the scope of a variable
based on its actual use, instead of where the user defined it. So,
@@ 162,9 +155,57 @@
scope. Also prune out useless variables, which should include field
variables after the translation to midil.
+test/vrkcomp2.diderot: Add support for code like
+ (F1 if x else F2)@pos
+This will require duplication of the continuation of the conditional
+(but we should only duplicate over the liverange of the result of the
+conditional.
+
+[GLK:8] Want: nontrivial field expressions & functions.
+scalar fields from scalar fields F and G:
+ field#0(2)[] X = (sin(F) + 1.0)/2;
+ field#0(2)[] X = F*G;
+scalar field of vector field magnitude:
+ image(2)[2] Vimg = load(...);
+ field#0(2)[] Vlen = Vimg ⊛ bspln3;
+field of normalized vectors (for LIC and vector field feature extraction)
+ field#2(2)[2] F = ...
+ field#0(2)[2] V = normalize(F);
+scalar field of gradient magnitude (for edge detection))
+ field#2(2)[] F = Fimg ⊛ bspln3;
+ field#0(2)[] Gmag = ∇F;
+scalar field of squared gradient magnitude (simpler to differentiate):
+ field#2(2)[] F = Fimg ⊛ bspln3;
+ field#0(2)[] Gmsq = ∇F•∇F;
+There is value in having these, even if the differentiation of them is
+not supported (hence the indication of "field#0" for these above)
+
co vs contra index distinction
some indication of tensor symmetry
+Permit field composition:
+ field#2(3)[3] warp = bspln3 ⊛ warpData;
+ field#2(3)[] F = bspln3 ⊛ img;
+ field#2(3)[] Fwarp = F ◦ warp;
+So Fwarp(x) = F(warp(X)). Chain rule can be used for differentation.
+This will be instrumental for expressing nonrigid registration
+methods (but those will require covscontra index distinction)
+
+Allow the convolution to be specified either as a single 1D kernel
+(as we have it now):
+ field#2(3)[] F = bspln3 ⊛ img;
+or, as a tensor product of kernels, one for each axis, e.g.
+ field#0(3)[] F = (bspln3 ⊗ bspln3 ⊗ tent) ⊛ img;
+This is especially important for things like timevarying data, or
+other multidimensional fields where one axis of the domain is very
+different from the rest, and hence must be treated separately when
+it comes to convolution. What is very unclear is how, in such cases,
+we should notate the gradient, when we only want to differentiate with
+respect to some subset of the axes. One ambitious idea would be:
+ field#0(3)[] Ft = (bspln3 ⊗ bspln3 ⊗ tent) ⊛ img; // 2D timevarying field
+ field#0(2)[] F = lambda([x,y], Ft([x,y,42.0])) // restriction to time=42.0
+ vec2 grad = ∇F([x,y]); // 2D gradient
+
+representation of tensor symmetry
(have to identify the group of index permutations that are symmetries)
dot works on all tensors
@@ 179,13 +220,6 @@
BUGS =================
======================
test/read2vecs.diderot:
// HEY (BUG?) shouldn't it be a type error to load this 2D array of
// 2vectors into a 2D *scalar* field? Instead, get:
// uncaught exception Fail [Fail: Error in compiling lic.diderot]
// raised at driver/main.sml:31.3931.76
image(2)[] Vimg = load("../data/vorttest.nrrd");

test/zslice2.diderot:
// HEY (bug) bspln5 leads to problems ...
// uncaught exception Size [size]