Re: [Geotools-devel] feature model review review part 2

2007-09-06 Thread Andrea Aime
Justin Deoliveira ha scritto: > Hi Andrea, > > I am just now looking at the Feature part of your review. And here are > my comments. > > * Property exposing name and type from descriptor > > This is kind of tricky issue. I need to provide some context. Consider > for a moment two types of attrib

[Geotools-devel] [continuum] BUILD FAILURE: Geotools

2007-09-06 Thread Continuum
http://geo.openplans.org:9090/continuum/buildResult.action?buildId=127&projectId=1 - This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configurat

Re: [Geotools-devel] [Geoapi-devel] feature model review review part 2

2007-09-06 Thread Andrea Aime
Martin Desruisseaux ha scritto: > Justin Deoliveira a écrit : >> * ComplexAttribute.getValue() to List >> >> +1 on this one. A list is still more useful even if you are telling >> people that the order in the list may be random. > > The only comment I could make on this feature review is related t

Re: [Geotools-devel] feature model review review part 2

2007-09-06 Thread Andrea Aime
Gabriel Roldán ha scritto: > On Wednesday 05 September 2007 01:46:51 Justin Deoliveira wrote: >> Hi Andrea, >> >> I am just now looking at the Feature part of your review. And here are >> my comments. >> >> * Property exposing name and type from descriptor >> >> This is kind of tricky issue. I need

Re: [Geotools-devel] [Geoapi-devel] GeoAPI feature model review

2007-09-06 Thread Martin Desruisseaux
Le jeudi 06 septembre 2007 à 08:55 +0200, Andrea Aime a écrit : > >> * InternationlizedString > Hum, imho i18n is a topic for UI and reporting that should not touch the > classes. Button labels and the like are a topic for UI. They are not covered by InternationalString, since the labels are provi

[Geotools-devel] [jira] Created: (GEOT-1482) FeatureUtilities wrapGridCoverageReader method should insert "params" attribute into the Features also

2007-09-06 Thread Alessio Fabiani (JIRA)
FeatureUtilities wrapGridCoverageReader method should insert "params" attribute into the Features also -- Key: GEOT-1482 URL: http://jira.codehaus.org/browse/GEOT-

Re: [Geotools-devel] GeoAPI feature model review

2007-09-06 Thread Adrian Custer
Hey Andrea, My apologies to you if I seemed to be telling you the obvious. I am well aware that you understand many and perhaps all of the complexity of CRS() stuff; my comments are *not* for your education but rather as nuances and phrasings for your review hopefully to help you write or benefit

[Geotools-devel] [jira] Created: (GEOT-1483) StreamingRenderer renderRaster method parses a wrong attribute

2007-09-06 Thread Alessio Fabiani (JIRA)
StreamingRenderer renderRaster method parses a wrong attribute -- Key: GEOT-1483 URL: http://jira.codehaus.org/browse/GEOT-1483 Project: GeoTools Issue Type: Bug Component

[Geotools-devel] [continuum] BUILD FAILURE: Geotools 2.4.x

2007-09-06 Thread Continuum
http://geo.openplans.org:9090/continuum/buildResult.action?buildId=132&projectId=81 - This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configura

[Geotools-devel] Build broken on 2.4: please remind to run "maven clean" before to commit an API change

2007-09-06 Thread Martin Desruisseaux
In the following class: org.geotools.resources.coverage.FeatureUtilities Method signature (...) wrapGridCoverageReader(AbstractGridCoverage2DReader reader) has been changed to (...) wrapGridCoverageReader(AbstractGridCoverage2DReader reader, GeneralParameterValue[]

[Geotools-devel] [continuum] BUILD SUCCESSFUL: Geotools

2007-09-06 Thread Continuum
http://geo.openplans.org:9090/continuum/buildResult.action?buildId=133&projectId=1 - This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configurat

Re: [Geotools-devel] [Geoapi-devel] feature model review review part 2

2007-09-06 Thread Andrea Aime
Andrea Aime ha scritto: > Citing from my review: > > "The whole model makes it explicit that when you're asking for a value, > you don't get one, but a Property instead. The first reaction is, why > would I need to get metadata along with data, isn't the feature model > enough? The javadoc doe

Re: [Geotools-devel] [Geoapi-devel] feature model review review part 2

2007-09-06 Thread Gabriel Roldán
Hi Andrea, some comments inline > Citing from my review: > > "The whole model makes it explicit that when you're asking for a value, > you don't get one, but a Property instead. The first reaction is, why > would I need to get metadata along with data, isn't the feature model > enough? I feel you

[Geotools-devel] On handling multiplicity, why AttributeDescriptor exists

2007-09-06 Thread Jody Garnett
Andrea Aime wrote: > (18.32.12) aaime: Hum, it seem the feature model review triggered a good > chain reaction > (18.32.17) aaime: but Jody is missing from the mix, sigh > (18.32.34) aaime: I guess he's the only one willing to face the > data/metadata mix issue... > (18.32.59) aaime: btw, you're

Re: [Geotools-devel] On handling multiplicity, why AttributeDescriptor exists

2007-09-06 Thread Jody Garnett
Andrea Aime wrote: > I understand. Yet I feel we lack a middle ground allowing to represent > associations in an efficient way, the same way deegree did. (In my benefit/complexity table this whole multiplicity thing is a good tipping point) I still don't see why it could not just be an implementa

[Geotools-devel] Handling of GridCoverage jpeg images on 2.4

2007-09-06 Thread Jody Garnett
Is there a prameter we can use on the GridFormat so we can set to drag images into memory again? In general 2.4 is way faster - in specifics JPEG is always a problem since you need to read the whole file all the time. Cheers, Jody Andrea Antonello wrote: > After a chat in irc I understood that

Re: [Geotools-devel] On handling multiplicity, why AttributeDescriptor exists

2007-09-06 Thread Andrea Aime
Jody Garnett ha scritto: > Andrea Aime wrote: >> (18.32.12) aaime: Hum, it seem the feature model review triggered a >> good chain reaction >> (18.32.17) aaime: but Jody is missing from the mix, sigh >> (18.32.34) aaime: I guess he's the only one willing to face the >> data/metadata mix issue...

[Geotools-devel] [continuum] BUILD SUCCESSFUL: Geotools 2.4.x

2007-09-06 Thread Continuum
http://geo.openplans.org:9090/continuum/buildResult.action?buildId=136&projectId=81 - This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configura

Re: [Geotools-devel] On handling multiplicity, why AttributeDescriptor exists

2007-09-06 Thread Andrea Aime
Jody Garnett ha scritto: > Andrea Aime wrote: >> I understand. Yet I feel we lack a middle ground allowing to represent >> associations in an efficient way, the same way deegree did. > (In my benefit/complexity table this whole multiplicity thing is a good > tipping point) > > I still don't see w

Re: [Geotools-devel] gpx module progress

2007-09-06 Thread Bolla Péter
Hi, In the case of a waypoint, adding a new attribute is fine, but a track may have separate timestamp for each node, so an attribute should hold a list of timestamps. (Which might be a good idea, or might be not...) As the gpx file itself stores the tracks as series of waypoints, not only a t

[Geotools-devel] [continuum] BUILD FAILURE: Geotools

2007-09-06 Thread Continuum
http://geo.openplans.org:9090/continuum/buildResult.action?buildId=135&projectId=1 - This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configurat