WONTFIX for 7.20
================
- look at:
http://www.eggheadcafe.com/software/aspnet/35467981/programmatic-console-appl.aspx
possibly better than nip2-cli.exe
- turning a column from Many Stats into a vector for doing arithmetic is very
tricky argh
add a matrix->vector converter? or maybe a one column or 1 row matrix should
also be a vector
- try:
nip2 Workspaces.uniformshapes2.A1.start_line=21 uniformshapes2.ws
--set does not work for non-toplevels
- try:
A1 = [1]
A2 = [x :: x <- A1]
change A1, A2 does not update, argh
we get:
link_expr_new: expr A2.$$lcomp0 references link->child = A1
so perhaps we are updating the local of A2, but not A2?
A2 is certainly being marked dirty ... on change of A1 we get:
row_dirty_set_single: A1 clear_error = true
symbol_dirty_set: A1 (0x1a59480)
symbol_dirty_set: A2 (0x1a595a0)
symbol_recalculate_check: untitled.A1
row_dirty_clear: A1
row_recomp_all: done row A1 - 0.000143873s
row_dirty_set_single: A1 clear_error = false
row_dirty_clear: A1
symbol_dirty_clear: A1 (0x1a59480)
success: [2]
symbol_recalculate_check: untitled.A2
symbol_dirty_clear: A2 (0x1a595a0)
success: [1]
so maybe A2.something is being updated, but the row is not
we now mark a row dirty if a sub-expr is dirty. but in row_renerate(), we
don't build subexprs
should we mark the subexpr dirty? (or maybe we do?)
or should we always copy all subexprs when we copy an expr
or only subexprs with no row?
do we calc rows outside-in or inside-out? does this affect copying subrows?
when do we copy now, the first time a row is made?
- we destroy and rebuild all links during recomp (eg. turn on DEBUG in
link.c), why is this? can't we only rebuild on a change of source text?
- fix the FIXME in itext_clear_edited() or wherever it is
- try:
start nip2
dir untitled
create A2, A3, etc.
A1 does not update
when we add/remove a def to workspace, should we mark the ws dirty?
- lambdas should allow patterns? eg.:
map (\[x, y] x + y) [[1, 2], [3, 4]] == [3, 7]
- OS X bundler:
http://sourceforge.net/apps/trac/gtk-osx/wiki/Bundle
test?
- imageinfo_make_paintable() no longer copies to a file, since this used to
cause problems with dangling pointers because of the im_close()s we had to
do
however, this means we now do all painting in memory :-( do we need to add
API to change a memory image (eg. a "p") into a file?
- test Joe's layout thing, compare to the thing we do in study2 to make the
diagnostic image
- im_blend(), im_ifthenelse(), im_add() etc. now do bandalike/formatalike
where do we use our bandalike/formatalike stuff? remove our stuff, though
make sure we have equivalents in vips now
- outline text example
- needs a custom convol menu item which can loop over a group of matricies
with a single image
actually, we need to nail this down, otherwise when we pass in a list of
sample texts the loops don't nest
- right-click menu on row button should have items for "Jump to referrer / WC1
/ JC1 ..." and "Jump to referred / ..."
- why didn't im_copy_file() work? mysterious
- line colours are wrong, argh, very mysterious, see
plot_new_gplot()
- gtk3.0 tests:
build with
#define G_DISABLE_DEPRECATED
#define G_DISABLE_SINGLE_INCLUDES
#define GDK_DISABLE_DEPRECATED
#define GTK_DISABLE_DEPRECATED
#define GDK_DISABLE_SINGLE_INCLUDES
#define GTK_DISABLE_SINGLE_INCLUDES
paste into config.h, somehow
need to remove:
GtkType
gtk_type_new
gtk_signal_connect
GTK_SIGNAL_FUNC
gtk_signal_handler_block_by_data
` gtk_signal_connect_object
GTK_CHECK_CAST
GTK_CHECK_TYPE
GtkSignalFunc
- add Set menu to Math with mkset/union/intersection/difference ?
could do bit operations on images?
- lcomps like:
argh = [x :: x <- poop]
the 'x' gets copied inside the lcomp, leaving a zombie 'x' attached to argh,
which Program / View / Errors then reports
fix: don't resolve names as we parse and junk the ugly patch list thing
instead, have a separate resolve stage that runs after we've moved scraps
of graph to their final home
- if we want full VipsObject introspection we will need a lot more
vips_object_arguments (name2gtype "VipsInterpolateYafrsmooth")
-> ["sharpness"]
need some equivalent for GParamSpec / VipsArgument
VipsArgument name type .... = class {}
return a list of these from vips_object_arguments()?
- heap_map_dict() should be reduce_map_dict(), since it does reduction, argh
redo heap_map_dict() in terms of reduce_map_dict()
actually, remove all the reduce_ stuff, it's daft to have a distinction
something to do when we break Snip out into libsnip
- Plot window should have image header menu item?
not trivial, image header needs a conversion to watch
we'd need to make a conversion in plotmodel
- filesel guess-file-type-from-suffix needs fixing
copy the vips model of having a user_name which is just "Workspace" or
somesuch, and making "Workspace file (*.ws)" string at runtime
use this to identity file types in util.c as well: get_image_info() needs it
- for rows made by typing stuff, always show the formula as well as the value
by default anyway?
we'd need to always show the up/down arrows, not just for classes
- drag from an image thumbnail to the ws background and you get a new column
with "A2" or whgatever in
does not work for plot thumbnails! how annoying
- right-click on image background to get a context menu with
save/replace/header? same as row thumbnail context?
- look at using goffice instead of gtkplot for graphs
http://ftp.gnome.org/pub/gnome/sources/goffice/
also in synaptic
there's also a new cairo-based gtkplot in SVN, apparently
- try
last [1..]
then CTRL-W ... we can quit the app, but it's still evaling and the prompt
never comes back
- Math / Cluster is a bit useless, will it work for complex numbers? vectors?
colours? groups?
what would we have to do to get it to work for these other types?
- toolkit load is where compiled code would go in
need to make load / parse / compile self-contained ... the only output is a
list of symbols, each with code and sub-symbols; there are no toolkits or
whatever made
after load / parse / compile, we need to walk the symbol list building tools
and all that stuff
we do:
load toolkit:
get filesize, date last modified, md5sum
look in ~/.nip2-7.x.x/cache for a file named with that md5sum
if present, open and check first two fields: filesize and date
if match, load compiled code
if no match, load / parse / compile toolkit, then save compiled
code to ~/.nip2-7.x.x./cache
walk symbol list building tools and all that stuff
how much time will this really save? can we easily get an estimate?
steps to follow:
1. make load / parse / compile self-contained, with separate pass to
build tools etc.
this is a useful cleanup whatever else we do
2. now we know exactly what the output of load / parse / compile is, we
should be able to write to a file
make our own binary format, don't use XML ... we want speed
3. try loading and benchmarking
can we junk the splash screen?
4. if the benchmarks look promising, harden and polish
- numbering of group of group save seems to skip one at end of line?
- Math / Cluster is a bit useless, will it work for complex numbers? vectors?
colours? groups?
what would we have to do to get it to work for these other types?
- segv in test_toolkits on laptop (inside fftw3) ???!? valgrinds cleanly on
work machine
- try
last [1..]
then CTRL-W ... we can quit the app, but it's still evaling and the prompt
never comes back
- configure no longers sets GMSGFMT, is this OK? test on OS X
- for rows made by typing stuff, always show the formula as well as the value
by default anyway?
we'd need to always show the up/down arrows, not just for classes
- drag from an image thumbnail to the ws background and you get a new column
with "A2" or whgatever in
does not work for plot thumbnails! how annoying
- right-click on image background to get a context menu with
save/replace/header? same as row thumbnail context?
- look at using goffice instead of gtkplot for graphs
http://ftp.gnome.org/pub/gnome/sources/goffice/
also in synaptic
WONTFIX for 7.14
================
- quit while a thumbnail is painting: IMAGEs are leaked? seems esp. bad with
greyc, perhaps just because it's so slow
- do we enforce no-args-to-LHS-pattern anywhere? try
[a,b] a b = 12;
- use destroy_if_destroyed() in more places?
grep destroy_ *.h
- after pressing "Process" in the edit window, we always select last_sym,
which is often not what we want
make it jump less after a process ... eg. try editing something in the
middle of Image/Transform, very annoying
- use compile->last_sym to spot chains of defs
multiple definitions of the same function are allowed, provided they all
multiple definitions of the same function are allowed, provided they all
have the same number of arguments, and provided only one of them has no
argument pattern matching
example:
fred [a, b, 1] = a * b;
fred (Image x) = rot90 x;
fred z = error "what";
any of these can have locals, those locals just apply to that one
definition
this compiles to:
fred $$a4
= $$fred1, is_list $$a4 && len $$a4 == 2 && $$a4?2 == 1
= $$fred2, is_instance_of "Image" $$a4
= $$fred3
{
$$fred1
= a * b
{
a = $$a4?0;
b = $$a4?1;
}
$$fred2
= rot90 x
{
x = $$a4;
}
$$fred3
= error "what";
{
z = $$a4;
}
}
so each pattern-matching definition generates a condition and an action
constants in patterns become part of the condition test
the action goes into a private function, the conditions are joined together
in the function wrapper
the no-pattern case (if present) becomes the action for the "otherwise"
clause in the wrapper
if not present, we generate (error "pattern match failed") or somesuch for
the default case
we will need to regenerate the wrapper function every time a definition of
fred is added or removed ... can we do this easily?
when are two definitions considered equal? should we warn about this? "fred
x" could occur in two files, for example
process:
* we see a "fred arg-list" incoming
* does the arg list contain any patterns?
yes:
* do we already have a fred in this scope?
yes:
* the existing fred must be the wrapper, this must be
a new possible RHS
* check that the number of args matches
no:
* generate a fred to act as the wrapper
* add args called $$arg1 etc. to the main fred
* add this new fred as a $$fredn local to the current
fred
* expand the patterns as local references to the main
fred's arguments
* parse the rest of the def in that context
* keep the pattern list around, we'll need it to generate the
ifs for the wrapper later
no:
* do we have a fred in this scope?
yes:
* check the previous fred was a pattern matcher
* check the number of args matches
* check there isn't already a default case
* add as above
no:
* add as a regular non-pattern definition
issues:
* where do we store the pattern lists? we can't expand them at
parse-time, since we need them to make the wrapper (which we can't
make until we've seen all the candidate RHS)
* when one of the RHS is changed, we need to regenerate the wrapper,
how do we do this? (could nuke the generated code in the compile
when we see a new RHS, then rebuild the wrapper in compile on the
next heap_copy?)
* our current condition generator won't work ... we need to test
consts as well, and it'll be rather inefficient as we'll repeatedly
test the trunk as we loop over the leaves --- instead, walk the
pattern recursively top-down testing each node
process:
* see "fred" (as opposed to simple_pattern)
* is there already a fred in scope?
yes:
* was current_compile->last_sym also a "fred"?
yes:
* another definition
yes:
* this must be an alternative definition
- we put the 2nd fred in as a local of the first, but then the 2nd can see all
the stuff the first has as locals
z = 42;
fred 1 = 12 { z = 99; }
fred 2 = z;
"fred 2" will return 99 :(
we need to make "fred 2 = z" into another fred at the same level, eg
$$alternate42$fred 2 = z;
Nope, then how do we link the freds together for remove etc.?
Better:
see a new sym (fred), create it
parse args with simple names becoming params, patterns becoming
$$arg42 plus a local $$patt42 holding the pattern source
expand the pattern to an access def as well, so our children can bind
to it
at the = sign, test for any pattern args present .. if there are none,
carry on as before
otherwise, make a new local called $$alternate42 or whatever and parse
the RHS into that
at the end of parse, need to resolve outwards twice, since we nest in
twice
now if we see another fred, check that the number of args matches and
then parse in as $$alternate99
what abut
fred 2 a = 12;
fred 1 b = 32;
the first fred will make a top-level with
fred $$arg12 a
=
{
$$alernate42 = 12;
$$patt12 = 2;
}
then when we parse the 2nd fred the name of the 2nd param is wrong :(
Even betterer:
use GLR to split off the four cases for us
ident =
pattern =
ident ident_list =
ident pattern_list =
change pattern syntax so that ident is not part of simple_pattern
need to change lcomp as well
so we need to check why PARSE_PARAMS gets used: can we do without the
params part? yes, it's used so we can edit functions, but we no longer
do this
these days all we need is expr I think, but we'd need a small action
wrapper around it to wipe out any existing tree and locals
Find_item = class
Menuaction "_Find"
("find a transform which will map sample image onto " ++
"reference") {
action reference sample = class
Transform b reference.width reference.height {
_vislevel = 3;
// controls
order = rubber_order;
interp = rubber_interp;
wrap = rubber_wrap;
max_err = Expression "Maximum error" 0.3;
max_iter = Expression "Maximum iterations" 10;
// transform
[a,b,c] = transform_search max_err max_iter order interp wrap
sample reference;
transformed_image = Image a;
final_error = c;
}
}
fails with
Bad superclass.
Superclass constructor
"Image_transform_item.Image_rubber_item.Transform"
should have no secret arguments.
but this:
Find_item = class
Menuaction "_Find"
("find a transform which will map sample image onto " ++
"reference") {
action reference sample = class
_t {
_vislevel = 3;
// controls
order = rubber_order;
interp = rubber_interp;
wrap = rubber_wrap;
max_err = Expression "Maximum error" 0.3;
max_iter = Expression "Maximum iterations" 10;
// transform
[a,b,c] = transform_search max_err max_iter order interp wrap
sample reference;
transformed_image = Image a;
_t = Transform b reference.width reference.height;
final_error = c;
}
}
(ie. make the superclass constructor into a member) works fine
- try using bison's location system
http://www.gnu.org/software/bison/manual/html_mono/bison.html#Locations
- add something to Symbol:
Symbol *access_for;
links generated access def to the $$thing4 which holds the RHS
handy for the program window, also maybe for lcomp code gen?
also for row edits
- don't offer to clear temps if there's been a crash
need to be able to test for process-still-running by PID
try
http://msdn2.microsoft.com/en-us/library/ms886766.aspx
gboolean
process_running( int pid )
{
HANDLE handle;
if( (handle = OpenProcess( 0, FALSE, pid )) ) {
CloseHandle( handle );
return( TRUE );
}
return( FALSE );
}
- autoarrange after every column resize or move
animate movement, so columns slither out of the way and in to place when
you drop
duplicate column placement would be odd: maybe place duplicate below? also
new column? what about dropping an image on to the ws background?
- there's some left-recursion in the parser, eg. comma_list, is this easily
fixable?
- add (*) (operator sections) ... need a binup / uop production? can't
do this without losing precedence stuff, it'll need a separate production
for sections
- magic definition maker could make a workspace-local def, rather cool
- test classmodel_dict_new() ... part of classmodel member automation
need to implement member edit for OPTION groups
classmodel_done_member() ... read widget set -> model
classmodel_buildedit_member( ... model -> build widget set
part of the [["key",value]] arg type
- can we get VIPS errors reported in Error too?
we'd need to add logging to vips I think
- toolkits / find doesn't find builtins on their name ... eg. search for
"im_add"
too hard to fix with the way searching is done now
- turn on update regions during drag, fix the x pos, try dragging, horrible
flickering as we update twice, once after the drag motion and once after the
recomp
if you comment out the explicit vobject_refresh() in
regionview_model_update() the flickering goes, but region dragging is then
very unresponsive
fix this when we get fast recomp back again
- have a test_types.ws ... test arithmetic on all combinations of _types?
- panner would be cool
- tooltips on Expression rows always show unedited formula
could special-case formula for things with expression RHS?
- what about iimage, iregion, iarrow ... can we member automate these? why are
they different?
- can't see error indications in noedit mode
should set a red background for display area as well as for rowview button?
- need to be able to override cons to be able to make a List class :-(
see reduce.c:1710
this will change the strictness of cons ... how much breakage will this
cause? very unclear
try this as a quick hack
need to do this before we can finish List
need List to make gamma easy
- unselected column headers are too like the bg colour on windows?
- python uses z.real and z.imag to extract real/image, should we add this too?
we don't really have complex as a true class, so it would be rather odd
need to add "[a].head" etc as well
- python blocks complex->real with casts ... insists you use .imag/.real or
abs()
- if nip sees a IM_RW_IMAGE argument, it could automatically do this:
int
im_flood_blob_copy( IMAGE *in, IMAGE *out, int x, int y, PEL *ink )
{
IMAGE *t;
if( !(t = im_open_local( out, "im_flood_blob_copy", "t" )) ||
im_copy( in, t ) ||
im_flood_blob( t, x, y, ink, NULL ) ||
im_copy( t, out ) )
return( -1 );
return( 0 );
}
so it would turn a single IM_RW_IMAGE arg into a paired input and output
arg
could make im_lineset() into a regular inplace func and rely on nip to wrap
and unwrap
junk flood_blob_copy
nip could do this lazilly ... if we see the user doing
im_line (im_line ...) ...
then we could make one memory image and call im_line twice on it
destructively ... cool! we'd need to check refcounts to make sure the
intermediate wasn't being used anywhere else
hmm! might actually be very hard, we don't have true refcounts for things in
the heap
need to do it on read instead:
- for image i
- use as an IM_RW_IMAGE arg ... copy to a memory area and pass in memory
handle
- return memory area IMAGE j, and set a flag saying "can operate on
destructively"
- if we use j as an IM_RW_IMAGE arg, skip the copy and just pass memory
area in destructively ... we now have two ImageInfo sharing a single
IMAGE
- !!!!
- does ImageInfo allow IMAGE sharing? not sure it does
- maybe this needs to be a vips8 feature when we'll have refcounts on
IMAGE
- tooltip on column says which other columns items in this column refer to,
and which columns refer to items in this column
- how about a nip start folder common to all versions
so nip2-7.11.14 tries
.nip2-7.11.14/start
.nip2-7.11/start
.nip2-7/start
.nip2/start
or maybe
.nip2/7.11.14/start
.nip2/7.11/start
.nip2/7/start
.nip2/start
bit less cluttered
also, we could have
.nip2/tmp
and not have multiple nip2 tmp areas
workspace recover after crash could break though ... maybe keep ws saves in
.nip2/7.11.4/tmp?
- think again about class arg checks
is there some way we can avoid the _check overhead? or at least check less
often
- plotpresent/imagepresent could have a common base class with the focus stuff
in? also kb nav, zoom, drag-scroll
a bit difficult, because we want two different policies on window resize:
plot should change the object to match the window
- photographic negative should also be in image/levels ?
no, it does ->sRGB, (255-) etc., so it's better as a filter
- gtk+ 2.12 has a treeview widget with rectangular select and grid lines
use instead of gtksheet?
- stop image flickering on clock recomp?
want background pattern to be a property of the image display widget, not
the image?
so we fade in tiles when that section of the image has never been displayed
before (eg. on scroll or zoom)
we don't fade when that section has been painted and we are just changing
the image (eg. on recalc)
if fadesteps == 1, only paint the sections of the tile for which mask == 255
this way we will never paint the bg pattern
need some hack for scroll/zoom ; test for mask == 255 would be slow :(
