-
Notifications
You must be signed in to change notification settings - Fork 3
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
CSIS URI variables in praxis 4 [RA/IA]: Heat mortality risk/impact #46
Comments
@p-a-s-c-a-l , @therter : please note that updating of the existing heat hazard resources with resources that use variables is more important than you think. Right now, we have over 100 resources linked to "European Wide Data Package" https://csis.myclimateservice.eu/node/682. That is completely unmaintainable: that is: it's nearly impossible to double-check if any of the resources is missing and if all of them are correctly defined. Not to mention updating anything. |
There is a mix of various capitalizations for the variable values in the links. Not sure if that's the reason for the error, but it doesn't look correct to me: filter=STUDY_VARIANT%3D’BASELINE’&filter=TIME_PERIOD%3D’Baseline’&filter=EMISSIONS_SCENARIO%3D’Baseline’&filter=EVENT_FREQUENCY%3D’rare’&filter=SZ_ID%3D2846 - ok I also wonder if the single quotes can be written as ' or must be written as %27? |
I looked into https://service.emikat.at/EmiKatTst/api/scenarios/2846/feature/view.2975/table/geojson?rownum=1000&filter=STUDY_VARIANT%3D’BASELINE’&filter=TIME_PERIOD%3D’Baseline’&filter=EMISSIONS_SCENARIO%3D’Baseline’&filter=EVENT_FREQUENCY%3D’Rare’&filter=SZ_ID%3D2846 and the output seems to be wrong to me. I also tried with 'rare', the result is the same - this filter seems to be ignored. using %27 instead of ' doesn't seem to have any effect. In fact, it would seem that other filters are ignored too. E.g., I'm also getting this out of it: |
does seem to work a bit better, so at least here it could be the quotes. (I replaced ' by %27). But the result is empty: |
@humerh any news on that? Could you please post a REST API URL that actually works so that i can test my implementation in clarity-h2020/simple-table-component#5 ? |
This should be solved now. Sorry. |
This works now: Thus, this must work too: |
I have a problem with map. Looking at the #46, I would expect that the same link as there must work here, just "clarity:view.2974" replaced by "clarity:view.2975" however, ah, OK - last parameter is too much. So this works:
And thus, this must be OK too:
|
Status: https://csis.myclimateservice.eu/node/1356 has been updated, for potential issues see #45 (comment) |
You can test the JSON API response with the new simple table component here (need to be logged-in in CSIS in the same browser!). You can even change the URL Template to test the other APIs like Population Exposure, but beware of the ${} pitfall. |
This should be OK now no? |
OK |
Update EMIKAT JSON API URIs has described here |
done |
Related to #42, #45, #46, #47, #49
I have implemented "Heat mortality risk/impact screening", please take a look
https://csis.myclimateservice.eu/node/1356/edit
@humerh: Are the uris correct?
@p-a-s-c-a-l: Is this how we want the variables used?
@therter: I hope that you can support this in the map soon...
The text was updated successfully, but these errors were encountered: