Hei Sunburned,
thats the point .. nobody could yet figure out how it happend and as you
read from Larrys post, the double rendering has almost no effect on
speed. So the clear option from my side is not to touch working code, if
not vividsolutions or Jon does it.
I don't know if you know that programmers saying:
"Don't touch a running system"
=> i unfortunately did not stick to that, talking not about OJ here, and
finally broke code.
I have actually some ideas to improve OJ concerning functionality
(Visualisation tools, build in raster support, map algebra,
on-the-fly-topolgy, projections, printing...) .. But from my side the
core is almost fit for GIS endusers, if talking not about specific apps,
e.g. multigeometry features, or the all-in-memory handling. Thus, i
sometimes wonder a bit on your ideas of improvements - instead working
directly towards functionality or on user manuals.
thanx again to you all for testing and discussion
stefan
Larry Becker schrieb:
> Well, no one knows how the second paint is happening. This is highly
> nonlinear code.
>
> Larry
>
> On 11/29/06, *Sunburned Surveyor* < [EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>> wrote:
>
> Thanks for your help with this Martin, Larry and Francisco.
>
> Is there any reason why we can't just remove the second call to paint
> in the BasicStyle class? We could do this in OpenJUMP's code base and
> then send the patch to Vivid.
>
> If everyone agrees I can make the change.
>
> SS
>
> On 11/29/06, Martin Davis <[EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>> wrote:
> > Agreed, that's almost certainly not it. The change would have
> been made
> > sometime in the last couple of months, I think.
> >
> > Larry Becker wrote:
> > > Last change I found in the renderer folder was:
> > > if (feature.getGeometry().isEmpty()) {
> > > changed to
> > > if (feature.getGeometry () == null ||
> > > feature.getGeometry().isEmpty()) {
> > > by Jon on 6/30/2006.
> > >
> > > Doesn't seem likely.
> > >
> > > regards,
> > > Larry
> > >
> > > On 11/29/06, *Martin Davis* < [EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>
> > > <mailto:[EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>> > wrote:
> > >
> > > Still, it would be nice to find out why BasicStyle.paint()
> is getting
> > > called twice. I took a look at this, but couldn't figure
> out why
> > > this
> > > was happening. (I have heard that Vivid has fixed this -
> maybe it
> > > would
> > > be worth looking at their codebase?)
> > >
> > > Larry Becker wrote:
> > > > Fco,
> > > >
> > > > Robert did some testing and we believe that although
> paint is
> > > > getting called twice, only one of the invocations is
> actually doing
> > > > work. You can observe this for yourself by opening a
> dataset large
> > > > enough to see it redraw. Only one actual draw occurs,
> even though
> > > > paint is called twice. This is not unusual in GUI
> programming. It
> > > > often happens that a repaint event can be issued that
> results in no
> > > > real work because the window update area has already been
> marked
> > > updated.
> > > >
> > > > regards,
> > > > Larry
> > > >
> > > > On 11/28/06, *fco lavin* <[EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>
> > > <mailto:[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>>
> > > > <mailto:[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>
> <mailto: [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>>>> wrote:
> > > >
> > > > larry, this...
> > > > i added a one line in class BasicStyle
> > > >
> > > > +-
> > > > public void paint(Feature f, Graphics2D g,
> Viewport viewport)
> > > > throws NoninvertibleTransformException {
> > > >
> > > > System.out.println (f); // added
> > > >
> > > > StyleUtil.paint(f.ge......
> > > > }
> > > > -+
> > > > then I draw, select the layerViewpanel, resize the
> windows
> > > or change
> > > > the windows (alt+tab) the result in console, with 3
> Features:
> > > >
> > > > [EMAIL PROTECTED]
> > > > [EMAIL PROTECTED]
> > > > [EMAIL PROTECTED]
> > > > [EMAIL PROTECTED]
> > > > [EMAIL PROTECTED]
> > > > [EMAIL PROTECTED]
> > > >
> > > > i test this in a linux, tomorrow i gonna test in
> windows and
> > > tell you
> > > > what i get.
> > > >
> > >
> > > --
> > > Martin Davis
> > > Senior Technical Architect
> > > Refractions Research, Inc.
> > > (250) 383-3022
> > >
> > > _______________________________________________
> > > jump-users mailing list
> > > [EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>
> > > <mailto:[EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>>
> > > http://lists.refractions.net/mailman/listinfo/jump-users
> <http://lists.refractions.net/mailman/listinfo/jump-users>
> > >
> > >
> > >
> ------------------------------------------------------------------------
> > >
> > > _______________________________________________
> > > jump-users mailing list
> > > [EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>
> > > http://lists.refractions.net/mailman/listinfo/jump-users
> <http://lists.refractions.net/mailman/listinfo/jump-users>
> > >
> >
> > --
> > Martin Davis
> > Senior Technical Architect
> > Refractions Research, Inc.
> > (250) 383-3022
> >
> > _______________________________________________
> > jump-users mailing list
> > [EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>
> > http://lists.refractions.net/mailman/listinfo/jump-users
> <http://lists.refractions.net/mailman/listinfo/jump-users>
> >
> _______________________________________________
> jump-users mailing list
> [EMAIL PROTECTED]
> <mailto:[EMAIL PROTECTED]>
> http://lists.refractions.net/mailman/listinfo/jump-users
>
>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> jump-users mailing list
> [EMAIL PROTECTED]
> http://lists.refractions.net/mailman/listinfo/jump-users
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
