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

Warnings for MWI flash-flood are identified by triggerValue 0 #1577

Open
4 tasks
gulfaraz opened this issue Sep 13, 2024 · 2 comments
Open
4 tasks

Warnings for MWI flash-flood are identified by triggerValue 0 #1577

gulfaraz opened this issue Sep 13, 2024 · 2 comments
Labels
dev This involves dev effort documentation Improvements or additions to documentation InDevOps Issue copy-pasted to Microsoft DevOps spike Something to talk about

Comments

@gulfaraz
Copy link
Member

gulfaraz commented Sep 13, 2024

Who requested this spike

@gulfaraz

Functional or Technical Spike

In flood (and probably other hazards), we identify warnings and triggers with a value greater than 0 - 0.3 for low warning, 0.7 for medium warning, and 1.0 for trigger. In this definition 0 is considered as no warning and no trigger.

In flash-flood (and typhoon), we identify warnings as place codes with a value 0, and triggers as place codes with a value 1.

User Story

As a dev
I want to use consistent logic across countries and hazards
So that I can solve user needs in understandable and maintainable code

Acceptance Criteria

  • Identify the definitions of warnings and triggers in the app
  • Fix inconsistencies and communicate changes to the relevant people
  • Document the decision in this issue and add this definition to the wiki
  • Create a follow up task to implement the changes

Timebox

This should take 30 mins to describe the problem, about 2 hours to research the existing implementation, 60 mins to agree on the new consistent approach.

@gulfaraz gulfaraz added documentation Improvements or additions to documentation invalid This doesn't seem right dev This involves dev effort labels Sep 13, 2024
@jannisvisser
Copy link
Contributor

@gulfaraz I found this spike in the customer success board. Shouldn't it be in product board?

@gulfaraz gulfaraz removed the invalid This doesn't seem right label Oct 14, 2024
@jannisvisser jannisvisser added refine This issue is ready for refinement and removed refine This issue is ready for refinement labels Oct 14, 2024
@jannisvisser
Copy link
Contributor

Refinement

  • Currently 2 different ways of identifying warnings (floods vs flash-floods/typhoon)
  • To fix this would be refactoring, for which there's probably no priority > create follow-up item still possible
  • But already valuable for documentation purposes
  • Create as-is vs to-be documentation (in this issue) > agree on form of documentation/diagram

@gulfaraz gulfaraz added the spike Something to talk about label Oct 15, 2024
@diderikvw diderikvw changed the title [SPIKE] Warnings for MWI flash-flood are identified by triggerValue 0 Warnings for MWI flash-flood are identified by triggerValue 0 Dec 2, 2024
@diderikvw diderikvw added the InDevOps Issue copy-pasted to Microsoft DevOps label Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev This involves dev effort documentation Improvements or additions to documentation InDevOps Issue copy-pasted to Microsoft DevOps spike Something to talk about
Projects
None yet
Development

No branches or pull requests

3 participants