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

IDEAS test new data integration #2708

Open
dmoglioni opened this issue Nov 20, 2024 · 7 comments
Open

IDEAS test new data integration #2708

dmoglioni opened this issue Nov 20, 2024 · 7 comments

Comments

@dmoglioni
Copy link
Contributor

As reported by @santilland all points made in the document (attached) have been addressed and made available at:
https://eodash-staging.eox.at/?catalog=ideas-data-addition

To be tested for production.

cc @aapopescu

PLES-TN-24-23, Data integration testing - IDEAS project v0.3_EOX_comments.docx

@AparicioSF
Copy link
Contributor

AparicioSF commented Nov 27, 2024

IDEAS testing is done. Thanks for integrating all the feedback to the dataset interactions! I would just add these 3 final comments which are minor on my opinion, and not that impeding of making it operational:

  • On Wildlife indicator, the indicator name on the left panel is called 'Indicator 2: Wildlife', maybe could be layer name could be updated to match the indicator.
  • Something that was discussed I believe, was to change the color scheme of different layers. E.g. on the HOUHP, each different layer, which have different information/metrics etc, they all have the same color pallet. Perhaps having a different one for each layer would make more sense - but nevertheless, I don't find a particular/major drawback
  • For the minesweeper game two things:
  1. It allows on the bottom to select dates - however, maybe this feature could be removed, as nothing happens, selecting different dates
  2. On the game description, I think the description could be clearer. Perhaps this could be achieved with adding additional information on - the goal: is it to flag ? And what's the actual meaning of left clicking? See what's there, but shouldn't we just click on it when we are 'sure' there is no 'bomb', e.g. places of high risk?

@santilland
Copy link
Collaborator

santilland commented Nov 27, 2024

  • Change Real Estate indicator naming to Heat Risk Indicator @santilland
  • @AparicioSF to provide small suggestions on minesweeper popop

@AparicioSF
Copy link
Contributor

AparicioSF commented Nov 27, 2024

Suggestions to update on the opening of the game:

  • The goal of the game is to UNCOVER (left-click) most tiles possible that do not have a 'mine', or in other words, tiles corresponding to areas with very high health risk. Your score is based on the percentage of uncovered area at the end of the game.

  • You can FLAG (right-click) the tiles with 'mines' (i.e. areas with very high health risk) to indicate they have a 'mine'.

  • Hint: Use visible layers to guide your exploration and uncover new tiles (left-click).

A new location is available daily — come back tomorrow to explore more!

@santilland
Copy link
Collaborator

Thanks for the inputs @AparicioSF it has been integrated now, i think this is now ready to be merged, @aapopescu what do you think?

@santilland santilland assigned aapopescu and unassigned AparicioSF Dec 3, 2024
@santilland
Copy link
Collaborator

Agreement on merging and ok to bring to production.
@AparicioSF to work on a short story to describe updates

@AparicioSF
Copy link
Contributor

@AparicioSF
Copy link
Contributor

AparicioSF commented Dec 17, 2024

I made some updates to this stories: (and still to-do updates):

  • respected the double cardinal for each chapter
  • intro to RACE (explain the context of IDEA and its indicators integration into RACE)
  • add image for the header of the story
  • add image of the dashboard view game
  • update all the links once the path images change (because they were added within the same branch where I'm writing this story: IDEAS_story (https://github.com/eurodatacube/eodash/tree/IDEAS_stories/app/public/data/storytelling-md)
  • Add hyperlinks to race page of each dataset within the part describing each indicator

Still there are these issues (maybe we can discuss on the upcoming meeting @santilland :

  • For some reason, the embedding of map tour (which I take from race.esa.int) doesn’t show on the story.
  • The images added are in the same folder of the story and using the same branch (IDEAS_story). However , @daniel is there a way of adding images, but does not forcibly require adding them to a branch? To avoid having to fix it to the new path, once the pull request is accepted, because the link to that image disappears (I don’t know if my question is understandable, and/or if you actually already told me how to do this, but I recall you indicating me about raw.githubusercontent.com…)
  • Finally, where on GitHub should I add here an entry for this story in particular? https://github.com/eurodatacube/eodash/blob/staging/app/src/config/stories.json , in any case I will do it once I know about the bullet before, because the link to the image that will head the story will disappear .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

No branches or pull requests

4 participants