ec-geolink / design

Design information about the EarthCube Geolink project.
8 stars 1 forks source link

Base ontology: Suggest adding new class 'SamplingMethod', adding new property 'hasSamplingMethod' for class 'PhyicalSample'. #71

Closed sparkji closed 8 years ago

krisnadhi commented 8 years ago

What's the difference between SamplingMethod and SamplingProcess? The PhysicalSample pattern from @YingjieHu already defines SamplingProcess and the relation (Physical derivedUsing SamplingProcess).

sparkji commented 8 years ago

'SamplingProcess' is subclass of 'Event' in PhysicalSample pattern. 'SamplingMethod' (CollectionMethod in SESAR) is not 'Event' but method by which a sample was collected, e.g. dreging, coring, blasting, etc.

I think they are different, please @YingjieHu https://github.com/YingjieHu make comment.

On Wed, Nov 18, 2015 at 1:34 PM, krisnadhi notifications@github.com wrote:

What's the difference between SamplingMethod and SamplingProcess? The PhysicalSample pattern that @YingjieHu https://github.com/YingjieHu already defines SamplingProcess and the relation (Physical derivedUsing SamplingProcess).

— Reply to this email directly or view it on GitHub https://github.com/ec-geolink/design/issues/71#issuecomment-157813239.

krisnadhi commented 8 years ago

@sparkji is there a controlled, external vocabulary for sampling method?

sparkji commented 8 years ago

We have consensus on revised "PhyscalSample" class within last meeting (DC/ECIP). I think you could ignore issues about samples I have proposed, just focus on IGSN kernel. Any comments, @robertarko https://github.com/robertarko?

On Wed, Jan 13, 2016 at 1:54 PM, krisnadhi notifications@github.com wrote:

@sparkji https://github.com/sparkji is there a controlled, external vocabulary for sampling method?

— Reply to this email directly or view it on GitHub https://github.com/ec-geolink/design/issues/71#issuecomment-171396845.

bob-arko commented 8 years ago

Agree. Goal now is to simplify GeoLink PhysicalSample class to align with IGSN schema: http://schema.igsn.org/

Focused on discovery.

Meanwhile we'll push further Sample details into other schemas , like SESAR local.

On Wed, Jan 13, 2016 at 12:49:11PM -0800, Peng wrote:

We have consensus on revised "PhyscalSample" class within last meeting (DC/ECIP). I think you could ignore issues about samples I have proposed, just focus on IGSN kernel. Any comments, @robertarko https://github.com/robertarko?

On Wed, Jan 13, 2016 at 1:54 PM, krisnadhi notifications@github.com wrote:

@sparkji https://github.com/sparkji is there a controlled, external vocabulary for sampling method?

— Reply to this email directly or view it on GitHub https://github.com/ec-geolink/design/issues/71#issuecomment-171396845.


Reply to this email directly or view it on GitHub: https://github.com/ec-geolink/design/issues/71#issuecomment-171428072

krisnadhi commented 8 years ago

My question was precisely because IGSN has a field "collectionMethod" whose values are from methodTypes. The IGSN XML schema seems to point to a restricted set of string values for method types.

bob-arko commented 8 years ago

Maybe don't need to restrict method in the GeoLink class. Just allow anyURI. But as a practical matter, many folks will populate it with controlled values from IGSN schema. ?

On Wed, Jan 13, 2016 at 12:56:41PM -0800, krisnadhi wrote:

My question was precisely because IGSN has a field "collectionMethod" whose values are from methodTypes. The IGSN XML schema seems to point to a restricted set of string values for method types.


Reply to this email directly or view it on GitHub: https://github.com/ec-geolink/design/issues/71#issuecomment-171429913

krisnadhi commented 8 years ago

anyURI or string? or both? http://schema.igsn.org/1.0/include/methodType.html provides strings though

sparkji commented 8 years ago

I think should be both anyURI and string. On Jan 13, 2016 4:03 PM, "krisnadhi" notifications@github.com wrote:

anyURI or string? or both? http://schema.igsn.org/1.0/include/methodType.html provides strings though

— Reply to this email directly or view it on GitHub https://github.com/ec-geolink/design/issues/71#issuecomment-171431601.

krisnadhi commented 8 years ago

Okay. Done!