Re: [JPP-Devel] Selectionstyle Patch

2010-07-30 Thread Larry Becker
Hi Matthias,

  Sorry I've been unable to get back to you.  I have been on travel, both
vacation and business, but I should be able to get back to my normal
schedule sometime next week.

regards,
Larry

On Fri, Jul 30, 2010 at 11:34 AM, Matthias Scholz m...@jammerhund.de wrote:

 Hi,

 please can Larry  or any other check my modifications again? Now I've
 solved the problem that the previous saved settings are restored from
 blackboard. The modifications in the AbstractSelectionRenderer should now
 smart enough, so that we have no performance problems and can set the
 properties with setter methods.

 It would be nice and important for my users if we can integrate the patch
 in the upcoming release.

 Regards

 Matthias

  Hi Larry,

 please see the attached file. There is one problem at the moment. After
 loading a file or project the AbstractSelectionRenderer use the default
 settings. So you must go to the options an click OK in the
 SelectionStyllingOptionsPanel. This is because during construction of the
 SelectionStyllingOptionsPanel instance there is no LayerViewPanel instance
 an so on no RenderingManager available. Is there a Listener which is fired
 if the RenderingManager is available or if a layer was added or a file was
 loaded???

 Regards

 Matthias

  Hi Matthias,

  I'm pretty sure that the right place to add setSelectionLineColor,
 setSelectionPointSize and
 setSelectionPointForm is AbstractSelectionRenderer since that is where
 the associated variables are declared.  You will have to remove /final /from
 HANDLE_WIDTH.  The real trick is getting the reference to
 FeatureSelectionRenderer to use the sets.  I think this can be done with
 RenderManager  by passing 
 FeatureSelectionRenderer.CONTENT_ID.toRenderManager.getRenderer().

 regards,
 Larry

 On Sun, Jul 11, 2010 at 7:58 AM, Matthias Scholz m...@jammerhund.demailto:
 m...@jammerhund.de wrote:

Hi Larry,

can we do the following?

- add some setSelectionLineColor, setSelectionPointSize and
setSelectionPointForm to
com.vividsolutions.jump.workbench.ui.renderer.Renderer interface
- implement simple setter in

  com.vividsolutions.jump.workbench.ui.renderer.FeatureCollectionRenderer,
wich is the parent of AbstractSelectionRenderer which is the parent of
FeatureSelectionRenderer, SelectionBackgroundRenderer,
PartSelectionRenderer, LineStringSelectionRenderer
- change the AbstractSelectionRenderer for using this variables while
paint()
- in a PlugIn in the OptionsPanel I can get the Renderer with
context.getLayerViewPanel().getRenderingManager().getRenderer()
call and
the I can change the values whith the setter

I'm not sure if the Renderer interface is the right place for the
setter
methods???

Please tell me your opinion, before I start. Any other suggestions are
welcome!

Regards Matthias

 I don't see an obvious solution.  For one thing, it is unclear how
 your plugin would affect the interoperability of
 SelectionBackgroundRenderer, PartSelectionRenderer, and
 LineStringSelectionRenderer which uses constructors to select
 different feedback mechanisms.

 Larry

 On Sun, Jun 27, 2010 at 3:00 PM, Matthias Scholz
m...@jammerhund.de mailto:m...@jammerhund.de
 mailto:m...@jammerhund.de mailto:m...@jammerhund.de wrote:

 Hi Sunburned Surveyor and Larry,

 sorry for my late reply, but i had no time to code last week :-(

 I agree with Larry's remarks. My question went toward the
concrete
 implementation, as I currently do not have enough knowledge
about the
 internals of OJ. At the beginning I would to build a plugin,
but I do
 not know how to tell the AbstractSelectionRenderer, how he
should
 render with my settings or with the old default one. So
actually I
 have
 no idea how to integrate the plugin in the rendering
process? At this
 point I need your help. I hope you understand my terrible
english ;-)

 Matthias

  I haven't looked at the code Mathias provided. It sounds like
 Larry's
  provided some great comments. I hope the changes can be
made so
  Mathias code can be integrated.
 
  If Mathias needs help making the mods Larry has suggested, he
 can let me know.
 
  The Sunburned Surveyor
 
  On Mon, Jun 21, 2010 at 6:28 AM, Larry Becker
 becker.la...@gmail.com mailto:becker.la...@gmail.com
mailto:becker.la...@gmail.com mailto:becker.la...@gmail.com
wrote:
 
  Hi Matthias,
 
  My concern about code efficiency is that this is a very low
 level routine in
  AbstractSelectionRenderer that is potentially called millions
 of times per
  session.  Everything possible (like blackboard access and
 object 

Re: [JPP-Devel] gps extension release candidate

2010-07-30 Thread Matthias Scholz
Hi Edgar,

I've tested gps.ext_0.2rc1.zip under Debian Lenny 32bit, OJ 1.3.1 G-Rays 
WBT 201 over Bluetooth. Nice work and truely usefull for me :-)

- You should check gps.ext_0.2rc1/lib/ext/rxtx/win32/Verkn?pfung mit 
win32.lnk, because until unzip I get a warning. No problem for me, but 
may be for endusers ;-)

- After starting OJ I get for about 80 seconds the OJ splashscreen 
without any hint what is OJ doing. I know that you scan for devices, but 
this should be done in background.

- The exception below is thrown if I disable the GPS Tracker checkbox. 
If I reenable it, GPS tracking runs.

Loading de.soldin.jump.gps.GPSExtension
Loading de.hawhamburg.sridsupport.SRIDSupportExtension
Loading net.refractions.postgis.PostGISConfiguration
GPS Extension is in: 
jar:file:/data/home/mscholz/openjump/bin/../lib/ext/gps_0.2rc1.jar!/
GPS (de.soldin.jump.gps) base is: 
/data/home/mscholz/openjump/bin/../lib/ext/gps_0.2rc1.jar
GPS Extension is in: 
jar:file:/data/home/mscholz/openjump/bin/../lib/ext/gps_0.2rc1.jar!/
GPS (de.soldin.jump.gps) libs are in: 
/data/home/mscholz/openjump/bin/../lib/ext/gps/
SwixML 1.5 (#149)
GPS Extension is in: 
jar:file:/data/home/mscholz/openjump/bin/../lib/ext/gps_0.2rc1.jar!/
RXTX Loader on: 'Linux' 'i386'
RXTX libraryPath is set to: '/data/home/mscholz/openjump/bin/../lib/ext'
Could not find: 
/data/home/mscholz/openjump/bin/../lib/ext/rxtx/librxtxSerial.so
RXTX succeeded loading: 
'/data/home/mscholz/openjump/bin/../lib/ext/rxtx/linux32/librxtxSerial.so'
WARNING:  RXTX Version mismatch
Jar version = RXTX-2.2pre2_loader
native lib Version = RXTX-2.2-20081207 Cloudhopper Build 
rxtx.cloudhopper.net
[ERROR][org.dinopolis.gpstool.gpsinput.nmea.GPSNmeaDataProcessor] - 
ERROR: Exception thrown on creation or processing of NMEA sentences:
[ERROR][org.dinopolis.gpstool.gpsinput.nmea.GPSNmeaDataProcessor] - 
$GPGSV,4,2,13,05,21,049,22,30,17,138,16,23,01,305,2
java.lang.IllegalArgumentException: Invalid NMEA Sentence (no '*'): 
$GPGSV,4,2,13,05,21,049,22,30,17,138,16,23,01,305,2
at 
org.dinopolis.gpstool.gpsinput.nmea.NMEA0183Sentence.init(NMEA0183Sentence.java:71)
at 
org.dinopolis.gpstool.gpsinput.nmea.NMEA0183Sentence.init(NMEA0183Sentence.java:104)
at 
org.dinopolis.gpstool.gpsinput.nmea.GPSNmeaDataProcessor.readMessages(GPSNmeaDataProcessor.java:335)
at 
org.dinopolis.gpstool.gpsinput.nmea.GPSNmeaDataProcessor.run(GPSNmeaDataProcessor.java:245)
at java.lang.Thread.run(Thread.java:619)

- Adding points from GPS is OK. For the future it would be nice if we 
have a automatic mode for adding point depending on time or distance.

- It is possible, that the connection to the device goes broken until  
OJ runs(Bluetooth problems etc).  The user should see a non technical 
message and not the

gnu.io.PortInUseException: Unknown Linux Application
at gnu.io.CommPortIdentifier.open(CommPortIdentifier.java:467)
at 
de.soldin.jump.gps.GPSTrackerPlugin.openConnection(GPSTrackerPlugin.java:528)
at 
de.soldin.jump.gps.GPSTrackerPlugin.actionPerformed(GPSTrackerPlugin.java:434)

for example.

- Under Linux it is a problem if you try to access the serial device 
more than once. So the Test settings button in the options should be 
disabled if GPS tracking is running.

- And finaly reprojection coordinates would be nice, but I know 



Regards

Matthias


 With some delay, I expected to have it done earlier, you might find an 
 updated version of the gps extension on sourceforge.

 http://sourceforge.net/projects/jump-pilot/files/p_GPS%20Extension/

 Features are:
 - use of serial gps devices (garmin,nmea,sirf)
 - tracking of selected tasks
 - creating/adding points from gps coordinate

 I'd appreciate anybody testing the extension and coming back with feedback or 
 bugs. A known limitation right now is that reprojection of gps coordinates is 
 broken (cts extension usage). This means coordinates from gps will for now 
 always be in the datum of the gps receiver is configured to output. I am 
 working it.

 .. kind regards ede




 --
 This SF.net email is sponsored by Sprint
 What will you do first with EVO, the first 4G phone?
 Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
 ___
 Jump-pilot-devel mailing list
 Jump-pilot-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
   


--
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share
of $1 Million in cash or HP Products. Visit us here for more details:
http://p.sf.net/sfu/dev2dev-palm
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net