-
Notifications
You must be signed in to change notification settings - Fork 13
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
As a traffic manager I would like to integrate "traffic events" of the regional roads in the Province of Bolzano in the Open Data Hub #307
Comments
@rcavaliere I think we can close this, right? |
@Piiit did you integrate the DC in testing environment (see above linked PR)? |
The related PR was automatically closed by Github, I will integrate it now (indirectly by myself, because I renamed a git branch) |
@rcavaliere The DC has only a event-uuid, but misses the event-series unique identifier... The event UUID is composed of:
Should I build the event-series-uuid with these fields?
|
@Piiit sorry I did not understand. We miss some fields to generate the event UUID, correct? In this case can't we use just these 4 fields to generate the UUID? |
we have two UUIDs here, one for the event itself and one for the event-series... since the second did not exist when the DC has been written, I ask, which fields form a single event, and which fields should be used to form an identifier for series of events? If this DC has only single events and no series, we can just use the same UUID (of all six fields) for event-uuid and event-series-uuid |
The source code uses all 6 fields at the moment, which should I take out? |
@rcavaliere The date interval seems to be wrong, I check that now |
You can test everything with https://mobility.api.opendatahub.testingmachine.eu/v2/flat,event/PROVINCE_BZ |
@Piiit ok let me test it in the next days... |
@rcavaliere Perfect, the date issues are solved now |
@Piiit I have tested the DC. I have the following open points:
I have already reported these points to @noctho. Please let me know, once checked / solved we can put the Data Collector in production |
It deals with the integration of this PR: #306
It depends on #263 since it needs the extension for integrating events type
The text was updated successfully, but these errors were encountered: