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

DCAT - definition of accrualPeriodicity #784

Merged
merged 4 commits into from Mar 12, 2019

Conversation

4 participants
@dr-shorthair
Copy link
Contributor

commented Mar 2, 2019

@agbeltran

This comment has been minimized.

Copy link
Member

commented Mar 5, 2019

I suggest we further clarify the definition from "The frequency at which dataset is published." to "The frequency at which dataset is published in the catalog" - to make explicit what is the collection is the catalog (with reference to the property definition "The frequency with which items are added to a collection").

@andrea-perego

This comment has been minimized.

Copy link
Contributor

commented Mar 5, 2019

@agbeltran said:

I suggest we further clarify the definition from "The frequency at which dataset is published." to "The frequency at which dataset is published in the catalog" - to make explicit what is the collection is the catalog (with reference to the property definition "The frequency with which items are added to a collection").

I'm a bit concerned about this proposal, as it would imply that a dataset must be in a catalogue. This is not always the case. Moreover, the update frequency concerns the data, so it's an intrinsic characteristic of the dataset, with no relationship with the catalogue(s) (if any) where the dataset is documented.

@agbeltran

This comment has been minimized.

Copy link
Member

commented Mar 6, 2019

Good point @andrea-perego - so, if we leave the text as it is, do you agree with the modifications in this PR?

@andrea-perego

This comment has been minimized.

Copy link
Contributor

commented Mar 6, 2019

@agbeltran said:

Good point @andrea-perego - so, if we leave the text as it is, do you agree with the modifications in this PR?

+1 from me.

@agbeltran

This comment has been minimized.

Copy link
Member

commented Mar 6, 2019

The point of datasets not in a catalogue is still a pending issue: #62

@agreiner

This comment has been minimized.

Copy link
Contributor

commented Mar 7, 2019

"This may be complemented by dcat:temporalResolution to give the rate at which items are added to a time-series." is ambiguous. (Is it added at publication, or added at data collection?) I think you must mean something more like "to give the time between collected data points in a time series". The two are only the same when the data are streamed.

@andrea-perego

This comment has been minimized.

Copy link
Contributor

commented Mar 7, 2019

I agree with @agreiner that temporal resolution is about data collection frequency. The proposed clarification makes sense to me.

@agbeltran

This comment has been minimized.

Copy link
Member

commented Mar 12, 2019

I have modified the statement about dcat:temporalResolution as suggested by @agreiner and agreed by @andrea-perego - I think with that the modification is ready and will merge the changes.

@agbeltran agbeltran merged commit 17c9169 into gh-pages Mar 12, 2019

DCAT revision automation moved this from To do to Done Mar 12, 2019

@agbeltran agbeltran deleted the dcat-issue728-simon branch Mar 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.