-
Notifications
You must be signed in to change notification settings - Fork 0
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
Implement new Adaptation Options table #77
Comments
@fgeyer16 : I suppose that only few people understand how this is even supposed to work, so please explain... I am looking at the https://csis.myclimateservice.eu/study/1/step/445/view/table step. That step shows https://csis.myclimateservice.eu/admin/structure/views/view/view_mode_for_gl_step/edit/page_6?destination=/study/1/step/445/view/table, which only says the following:
That table view mode is defined in https://csis.myclimateservice.eu/admin/structure/types/manage/gl_step/display/table and displays "table_app", which is defiend as external applicaiton. So far so good, but what I need is an internal view instead. There is such a view defiend for "twins" step, as shown here https://csis.myclimateservice.eu/admin/structure/types/manage/gl_step/display/twins?destination=/contextual/render&destination=/study/1/step/445/view/twins So, I could add a viewfield "view table" and set it per default to the view that will produce what we need, but how do I assure that this table is only shown in the "adaptation options" step, whereas the javascript app table is shown in other cases? |
Following the discussion with Florian, I know how to do this in principle now. |
List of changes to the system: In the study group:
In the gl_step node: |
More changes: Adaptation option effects node:
Adaptation options node:
"LocalEffectsParameters" and "Local hazard change" taxonomies
It's up to our colleagues from plinius and Heinrich to decide how they want to use this (if at all), but this allows to transparently add parameters for local effect calculations. And the local effect can be also directly on a hazard if that is what we want. |
Even more changes... Adaptation options table on https://csis.myclimateservice.eu/adaptation-options-overview reworked.
Adaptation options main view reworked:
Adaptation options edit mode improved
|
I was stalled with this due to work on templates for studies. Still on my TODO |
I have implemented a new "exposure twins" view and realised that the way we are looking for available twins is broken. I also implemented a view to show available AOs, but it will suffer from the same issues. |
@patrickkaleta : I have implemented a new "adaptation options" table for the study "ADAPTATION OPTIONS: IDENTIFICATION" step. see https://csis.myclimateservice.eu/study/17/step/1030/view/table Problem: the buttons are disabled. Why? |
It works now. Adaptation options can now be added to the GL-Step using the CSIS-helpers module. FYI: I could remove this "safety lock", but I'm not sure what will happen when the JSON:API suddently tries to add content types to the field_included_twins field that are not allowed in the field's configuration. When I find time, I will try this out, but until then I would leave the safety lock on. From my point of view this issue can be closed. @DenoBeno do you require anything else regarding this topic right now? |
@patrickkaleta : no, everything works now. the only remaining open issue is that Heinrich needs to know which adaptation option to apply to which types of surface layers. I suppose that we need a list or taxonomy of possible surface layers and a field in the AOs that can reference to 0..1 or 0..* such layers. If 0 - nothing to do for emikat. TODO
FYI Since local effect change accepts two types of entries AND we can't be sure that al lthe linked entries are correct, EMIKAT will need to to decide which ones are usable and ignore the rest. |
OK, I added the entries to taxonomy now, see https://csis.myclimateservice.eu/jsonapi/taxonomy_term/land_use?fields[land_use]=name,%20field_id I also added a field_land_use ("Applicable to") and filled in the values for some existign AOs. See https://csis.myclimateservice.eu/adaptation-options-overview This is done for me now. What's still needed is new entries/improving existing entries, but that's on PLINIUS and co. |
I thought that I'm finished with this, but @humerh has just reported that the "local effects change" can not be the same for all relevant land use categories. Of course, this is already possible to do but in a way that would lead to inflation of the adaptation options, which would make them more difficult to use for the end users. I need to think about this.... |
Since this is already ranged into "done", I'll close it again and open a new issue instead. |
We need a new adaptation option table to replace the one shown on https://csis.myclimateservice.eu/study/1/step/445/view/table
This table must have the following features:
The idea is to have something working now and to add a possibility to APPLY the adaptation options to the project and see the result later.
probably related to:
The text was updated successfully, but these errors were encountered: