Determine Risk Level for Exposure Windows & Aggregation (EXPOSUREAPP-3537,...
Determine Risk Level for Exposure Windows & Aggregation (EXPOSUREAPP-3537, EXPOSUREAPP-3518) (#1546) * Split and hide the protobuf config behind interfaces with individual mappers responsible for creating the desired formats. * Merge branch 'release/1.7.x' into feature/3455-more-frequent-riskscore-updates-configs # Conflicts: # Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/test/risklevel/ui/TestRiskLevelCalculationFragmentCWAViewModel.kt # Corona-Warn-App/src/main/java/de/rki/coronawarnapp/transaction/RiskLevelTransaction.kt # Corona-Warn-App/src/test/java/de/rki/coronawarnapp/transaction/RiskLevelTransactionTest.kt * Make the AppConfig observable. Provide the server time offset. Offer a lastUpdatedAt timestamp. Add an app config specific test screen. Clean up test screens a bit and move debug options out of API test options. * Fix test regression due to refactoring (moved code around). * Store the server timestamp and offset at retrieval. Switch to config storage via json to be able to store additional meta data fields (i.e. time). * KLint and Me have a hate relationship based on both mutual admiration. * Fix time offset parsing being locale dependent. * Fix broken unit tests. * Improve offset accuracy, move before unzipping. * Fix overly long livedata subscription to results (viewmodel scope vs observer scope) * Add mapping for the new protobuf configs + tests. * For cached (retrofit) response, we need to check the cacheResponse and its timestamps to determine an accurate time offset. * Exposure a boolean property to tell us when a fallback config is being used. * Hide the observable flow<ConfigData> behind a method that can automatically triggers refreshes. * Use a common mapper interface. * set old risklevelcalculation deprecated * Created skeleton for new risk calculation and aggregation * Implementing steps to aggregate results form exposure windows - wip * Address PR comments and KLints. * Fix refactoring regression. * ktlint * Added ExposureWindowRiskLevelConfig and ExposureWindowRiskLevelConfigMapper for new config api (not yet introduced) Signed-off-by:Kolya Opahle <k.opahle@sap.com> * Added first Implementation of exposure window based calculateRisk function Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Added generics to Range.inRange Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Added Ugly Hack to RiskLevelTransaction to allow for compilation during testing Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Linting and injecting RiskLevelCalculation into TestRiskLevelCalculationFragmentCWAViewModel, currently wont build because ExposureWindowRiskLevelConfig has no Provider Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Linting extravaganza Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Lint Wars Episode VI: Return of the trailing Comma * Improve config unzipping code. * Add flag to forward exception thrown during HotDataFlow.kt initialization. * Don't specify a default context via singleton. * Move download and fallback logic into it's own class just responsible for sourcing the config: "AppConfigSource". "AppConfigProvider" is now only responsible for making it available. * Simplify current concepts for making the app config observable until we have a default configuration. * Improve app config test screen, delete options, better feedback. Show toast instead of crash on errors. * Fixed GSON serialization not encoding/decoding the byte array correctly. Added specific type adapters for instant and duration to get cleaner json. * Remove type adapters from base gson due to conflict with CalculationTrackerStorage. * We want to default to forced serialization of instant by our converters, instead of using the default serialization which will differ between Java8.Instant and JodaTime.Instant, to prevent future headaches there, register explicit converters by default, and overwrite them if necessary (currently only needed for CalculationTrackerStorage.kt). * Improve AppConfigServer code readability by moving code into extensions. * Fix merge conflicts * Added missing import to WorkerBinderTest * fixed unit tests * Removed auto formatting on unrelated files (revert + cherry pick in other commit) Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Implementing steps to aggregate results form exposure windows * Renamed ExposureWindowRiskLevelConfig to ExposureWindowRiskCalculationConfig * adjusted & refactored Windows aggregation * removed example Values * satisfy lint * make Aggregation work with Instant now * Use long while calculation * Added normalizedTimePerDayToRiskLevelMappingList to AppConfig * normalizedTimePerDayToRiskLevelMappingList from AppConfig * satisfy lint * Get AppConfig on init and listen for updates * exposureData to aggregatedRiskPerDateResult * Corrected name in ConfigParserTest * use instant for specific aggregation logs * satisfy CI * satisfy detekt * exposure history exception & log adjustment * Fixed unittests for new config parser and risk levels Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Added some logging to the calculateRisk function and removed the suspend qualifiers as AppConfig is fetched during init Signed-off-by:
Kolya Opahle <k.opahle@sap.com> Co-authored-by:
Matthias Urhahn <matthias.urhahn@sap.com> Co-authored-by:
BMItter <Berndus@gmx.de> Co-authored-by:
harambasicluka <64483219+harambasicluka@users.noreply.github.com>
Showing
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/AppConfigModule.kt 13 additions, 5 deletions...in/java/de/rki/coronawarnapp/appconfig/AppConfigModule.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/ExposureWindowRiskCalculationConfig.kt 20 additions, 0 deletions...awarnapp/appconfig/ExposureWindowRiskCalculationConfig.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/mapping/ConfigMapping.kt 3 additions, 1 deletion...a/de/rki/coronawarnapp/appconfig/mapping/ConfigMapping.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/mapping/ConfigParser.kt 11 additions, 2 deletions...va/de/rki/coronawarnapp/appconfig/mapping/ConfigParser.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/mapping/DefaultConfigMapping.kt 5 additions, 2 deletions...i/coronawarnapp/appconfig/mapping/DefaultConfigMapping.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/mapping/ExposureWindowRiskCalculationConfigMapper.kt 44 additions, 0 deletions...nfig/mapping/ExposureWindowRiskCalculationConfigMapper.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/DefaultRiskLevels.kt 322 additions, 1 deletion.../main/java/de/rki/coronawarnapp/risk/DefaultRiskLevels.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/RiskLevelTask.kt 11 additions, 11 deletions.../src/main/java/de/rki/coronawarnapp/risk/RiskLevelTask.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/RiskLevels.kt 12 additions, 0 deletions...App/src/main/java/de/rki/coronawarnapp/risk/RiskLevels.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/result/AggregatedRiskPerDateResult.kt 10 additions, 0 deletions.../coronawarnapp/risk/result/AggregatedRiskPerDateResult.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/result/AggregatedRiskResult.kt 12 additions, 0 deletions.../de/rki/coronawarnapp/risk/result/AggregatedRiskResult.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/result/RiskResult.kt 10 additions, 0 deletions.../main/java/de/rki/coronawarnapp/risk/result/RiskResult.kt
- Corona-Warn-App/src/test/java/de/rki/coronawarnapp/appconfig/mapping/ConfigParserTest.kt 6 additions, 1 deletion...e/rki/coronawarnapp/appconfig/mapping/ConfigParserTest.kt
- Corona-Warn-App/src/test/java/de/rki/coronawarnapp/risk/RiskLevelsTest.kt 7 additions, 0 deletions...src/test/java/de/rki/coronawarnapp/risk/RiskLevelsTest.kt
Please register or sign in to comment