linking Long Stayers to the System Performance charts - #1026
Draft
alex-silverman wants to merge 18 commits into
Draft
linking Long Stayers to the System Performance charts#1026alex-silverman wants to merge 18 commits into
alex-silverman wants to merge 18 commits into
Conversation
…hardcode the values (15/60/90 days) for groups of project types into `days_lh_valid`. This change moves away from the hardcoded values and instead uses the long-stayer settings, which the user can manipulate.
alex-silverman
changed the title
alex-silverman
marked this pull request as draft
July 8, 2026 14:35
…ories datasets in 07, we create them as part of the observeEvent of the Local Settings, since this will run after the upload has been processed. This shouldn't affect the processing time and timing, as these datasets are still generated just once as part of the upload, but now after the upload, rather than during. And now, when they change the Local Settings it will recreate the datasets as needed. We now create `days_lh_valid` on the lh_info dataset. This is the soonest it's needed and also makes more sense as it is an LH-focused variable.
… into feat/link_long_stayers
And log those changes to the metadata so we know how often users are changing these
…atically - which may mean the user is forced to wait a logn time for System Performance to update even if they only care about DQ - we give the user the option to update the DQ and update the System Performance separately.
… into feat/link_long_stayers
…tem Performance, and also since we've renamed other tabs
… into feat/link_long_stayers
Merge branch 'feat/link_long_stayers' of github.com:abtassociates/eva into feat/link_long_stayers # Conflicts: # changelog.R
…s) and their _prev versions, to a function since we need a way to set them before we explicitly update the local settings.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Currently, we hardcode the values (15/60/90 days) for groups of project types into
days_lh_valid. This change moves away from the hardcoded values and instead uses the long-stayer settings, which the user can manipulate.See https://github.com/orgs/abtassociates/projects/1/views/1?pane=issue&itemId=93873643.