-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Time serie selection handled by the advanced metadata panel #9821
Comments
@mattiagiupponi that's too bad. I suppose GeoNode importer configures the dateformattransform transformation. The new importer shouldn't have this problem, since we publish from the DB and the field type should be automatically recognized. Do you confirm @mattiagiupponi? For the gs-importer case we only have two options I guess:
The automatic parsing of datetime fields on the BE could leverage the current dataset_eligible_for_time_dimension. It's not very robust, since it only read the first feature to guess the field format, but it could be probably improved a bit to make it more robust (if the first feature's value is empty try another one, maybe samples from the first 10 features). Since we're going to adopt the new importer for all the vector formats soon, I would consider solution 1 (if it doesn't take too much) |
The new importer will take the field type based on how ogr2ogr reads the dataset's schema. But I guess that we will continue to save the data as "Integer" and not as "date". I can try it on local to have an answer on this.
I think this is not strictly needed.
I would avoid too much guessing from the client side. |
@mattiagiupponi the selectbox should recognize Integer fields too IMHO. Consider the case when we have just the year as an instance. This should be also the current GeoServer behavior. |
Indeed @afabiani , this phrase is no longer valid:
|
The final proposal is the following. CURRENT IMPORTER: NEW IMPORTER:
Once the new time field will be available in Geoserver, the field will appear inside the TIme dimension management box under the GeoNode's dataset metadata settings (already implemented). |
…anel (#9840) * [Fixes #9821] Time serie selection handled by the advanced metadata panel * [Fixes #9821] add new attribute to time series * [Fixes #9821] add new attribute to time series * [Fixes #9821] add new attribute to time series * [Fixes #9821] add new attribute to time series * [Fixes #9821] add new attribute to time series * [Fixes #9821] add new attribute to time series * [Fixes #9821] add new attribute to time series * [Fixes #9821] add new attribute to time series * [Fixes #9821] add new attribute to time series * [Fixes #9821] Time serie selection handled by the advanced metadata panel * [Fixes #9861]Cannot open metadata detail with thesaurus installed * [Fixes #9821] Time serie selection handled by the advanced metadata panel * [Fixes #9821] Time serie selection handled by the advanced metadata panel * [Fixes #9821] Fix wrong subtype in case of timeserie dataset * [Fixes #9821] Fix circleci build
@marthamareal please test this on the development server. 1 - Test that the configuration is correctly shown when you upload a new layer and configure it as time series These tests must be added to Cucumber. |
The time selection is going to be added to the advanced metadata page in the
settings
panel. If thehas_time
is selected a new panel for the time series settings is open:Based on the selection of the
presentation
, two new input boxes are loaded to let the user configure the time series:The ? open a modal with some information about "how to configure" the time series
Internally the
dataset_metadata
view will communicate withGeoServer
to configure the time series. If thehas_time
is removed, also the configuration inGeoServer
is disabled.The text was updated successfully, but these errors were encountered: