Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Sites table, moving Lat/Lon to SamplingFeatures #152

Open
aufdenkampe opened this issue Aug 8, 2018 · 8 comments
Open

Remove Sites table, moving Lat/Lon to SamplingFeatures #152

aufdenkampe opened this issue Aug 8, 2018 · 8 comments
Assignees
Labels

Comments

@aufdenkampe
Copy link
Member

aufdenkampe commented Aug 8, 2018

Following ideas in #142 (comment) and below, I've concluded that we should fully merge the Sites table into the SamplingFeatures table, in agreement with @PleiadesAustralia, for a number of reasons. These include:

@aufdenkampe
Copy link
Member Author

Here is the diagram from 019bc37:
odm2samplingfeatures_odm2 1_dev_019bc37

@PleiadesAustralia
Copy link

PleiadesAustralia commented Aug 8, 2018 via email

@PleiadesAustralia
Copy link

PleiadesAustralia commented Aug 8, 2018 via email

@dr-shorthair
Copy link

dr-shorthair commented Aug 8, 2018

The current 2018 SiteType vocabulary is inappropriate to describe a SamplingFeature and more a list of FeatureOfInterest types.

Yes - I agree that the list is a mixture with a lot of feature-types that are not usually sampling features.

However, it is worth noting that classifying something as a sampling feature is primarily related to its role in relation to another (larger or inaccessible) object or feature - it is a sampling-feature if it is intended to be representative of something else. This role does not inhere in the observable characteristics or other functional classifications of the feature, merely in the intention that it is-a-sample-of some other feature. The same feature might be both a feature-of-interest and a sampling-feature, in different contexts. For example, a well might be of interest in its own right (for producing water) and might also be a sampling feature with respect to the aquifer that it intersects.

@horsburgh
Copy link
Member

@aufdenkampe - also, what about just Specimens that are SamplingFeatures, but have no latitude and longitude? This is why we separated Sites and Specimens from SamplingFeatures in the first place. In my old age I'm liking relational databases less and less because the representation of specialization is just silly.

Also - where did the link between FeatureActions and RelatedFeatures come from?

I'm assuming you're working in a new branch here? There's a lot of stuff that uses the existing construct.

@smrgeoinfo
Copy link

smrgeoinfo commented Aug 9, 2018

re @aufdenkampe FeatureAction links-- There needs to be a link to an action on a feature relationship to enable a process to be associated with derivation of one sampling feature from one or more parent SFeatures. Seems like having 'process' property on relatedFeatures that links to an Action would be a clearer solution.

@aufdenkampe
Copy link
Member Author

@horsburgh, good questions. Yes, all of this is in the new ODM2.1_dev branch. Also, because I'm addressing lots of different issues sequentially, I decided to repost all the proposed schema changes in a new Draft ODM2.1 schema for review issue (#153). So, as @smrgeoinfo pointed out, that new RelatedFeaturesRelationID resulted from work we did in Nov. 2016 for #130.

@horsburgh, note that Lat & Lon are now nullable, so a SamplingFeature.Specimen would not require Lat/Lon.
@PleiadesAustralia, for the notes for Lat & Lon, I had put "Latitude of centroid of SamplingFeature", but that's not viewable from these simple .png images.

@dr-shorthair, take a look at my proposed FeatureOfInterest additions in #132 (comment). I think the construct there and previously in ODM2.0 can accommodate what you described.

@PleiadesAustralia
Copy link

PleiadesAustralia commented Aug 9, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants