[GRASS-dev] xganim: fix compilation or remove completely?

[Vaclav Petras]

There are no features in xganim which g.gui.animation does not have. The
same applies to gm_animate.

The only exception is accepting wildcard expression instead of map name
which is than expanded into the list of map names but this is not problem
which this tool should be solving. In GUI you can use regular expressions
to input multiple maps or time series and in command line, there is the
g.mlist module.

xganim removed in r58484 and r58487.

Vaclav

[r58484] https://trac.osgeo.org/grass/changeset/58484
[r58487] https://trac.osgeo.org/grass/changeset/58487



On Wed, Dec 11, 2013 at 10:55 PM, Helena Mitasova <hmitaso at ncsu.edu> wrote:

> Vasek,
>
> I think that if there are no objections xganim can be removed from grass7,
> the new animation tool is much, much better.
>
> It may be worth keeping it in grass6.4*. To test xganim just use any
> output from r.sim.water or r.sun.daily or nags head series.
> If you are seeing window larger than your screen check your region
> settings - the size of the xganim window in pixels
> is the size of your region (so either change the region to lower
> resolution or smaller spatial extent).
>
> We can go through xganim in the lab (I may have it still running on the
> mac laptop) to make sure
> we have everything in the new animation tool that was in xganim, but I
> believe that we do,
>
> Helena
>
> Helena Mitasova
> Associate Professor
> Department of Marine, Earth, and Atmospheric Sciences
> 2800 Faucette Drive, Rm. 1125 Jordan Hall
> North Carolina State University
> Raleigh, NC 27695-8208
> hmitaso at ncsu.edu
>
> "All electronic mail messages in connection with State business which are
> sent to or received by this account are subject to the NC Public Records
> Law and may be disclosed to third parties.”
>
> On Dec 11, 2013, at 10:38 PM, Vaclav Petras wrote:
>
> > Hi,
> >
> > when testing the previous problems with C++11/clang/Mavericks, I used
> the following compilation settings on Ubuntu 12.04:
> >
> >   export CC=clang
> >   export CXX=clang++
> >   export CFLAGS="-ggdb -Wall -Wextra
> -Werror-implicit-function-declaration -Wmissing-prototypes
> -Wmissing-declarations -Wredundant-decls"
> >   export CXXFLAGS="-ggdb -std=c++11"
> >
> > I get the following errors in `visualization/xganim` directory:
> >
> > ./bitmaps/rewind.xbm:4:28: error: constant expression evaluates to 128
> which cannot be narrowed to type 'char'
> >    0x00, 0x00, 0x00, 0x30, 0x80, 0x01, 0x28, 0x40, 0x01, 0x30, 0xa0,
> 0x01,
> >                            ^~~~
> > ./bitmaps/rewind.xbm:4:28: note: override this message by inserting an
> explicit cast
> >    0x00, 0x00, 0x00, 0x30, 0x80, 0x01, 0x28, 0x40, 0x01, 0x30, 0xa0,
> 0x01,
> >                            ^~~~
> >                            stat)c_cast<char>(
> >
> > and a lot of other similar errors.
> >
> > I'm not getting these errors on Mavericks.
> >
> > I'm not able to test the xganim tool [1] right now; there is no
> ready-to-use example and I'm not sure which environment it requires (I'm
> getting white window bigger than my screen). However, the question is: It
> is worth keeping the xganim tool in code or should we remove it completely?
> >
> > Are there any functions which xganim tool does have comparing to
> g.gui.animation [2]? Or are there some other advantages of xganim? We
> should avoid loosing functionality as it was with tools such as gm_animate
> [3]. So these questions shall be answered before removing xganim.
> >
> > Best,
> > Vaclav
> >
> > [1] http://grass.osgeo.org/grass70/manuals/xganim.html
> > [2] http://grass.osgeo.org/grass70/manuals/g.gui.animation.html
> > [3] http://grass.osgeo.org/grass64/manuals/gm_animate.html
> >
> > _______________________________________________
> > grass-dev mailing list
> > grass-dev at lists.osgeo.org
> > http://lists.osgeo.org/mailman/listinfo/grass-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20131218/0ce6be54/attachment.html>