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  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.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
>>>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
>>>> mailto:becker.la...@gmail.com>
>>>>>
>>>wrote:
>>>> >

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.(NMEA0183Sentence.java:71)
at 
org.dinopolis.gpstool.gpsinput.nmea.NMEA0183Sentence.(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
https://lists.sou