lzim / teampsd

Team PSD is using GitHub, R and RMarkdown as part of our free and open science workflow.
GNU General Public License v3.0
15 stars 23 forks source link

wk4 june_epic MTL Manual Edits - Suicide Prevention and Stepped Care floors and episode counts #648

Closed holbrooa closed 3 years ago

holbrooa commented 5 years ago

@TomRust and @holbrooa are meeting to discuss emergent SP parameter issues.

  1. High-symptom proportions are limited to user-selected SMH team Currently, high-symptom proportions are very low, because patients are categorized as high-symptom based on being stepped up to the SMH team that the user selected. In real life, many patients are stepped up to other SMH teams, so categorizing those patients as low-symptom seems wrong. We propose defining high-symptom patients as those who are above 50% on the symptom distribution for this team. There would not necessarily be the same number of high- and low-symptom patients, but a new patient to the team would be equally likely to flow into either stock. This seems easier to explain than the current state, where we sometimes have to explain percentages as low as 2%. This is very little work on both the data and the model side. This change would necessitate changes on the sim UI - both i text and removing the symptom proportion entries in the team data table.

  2. Floors should be re-examined In light of almost all engagement durations being below the floor for recent data pulls, it seems like we should re-think the clinical floors and how they map onto the transfer logic. We went back and looked at meeting notes, and found that the floors may have been implemented incorrectly to begin with - the 24 week time to stabilize was applied everywhere. Also, we think PC/PCMHI should probably be exempt from most or all of these floors, because they don't necessarily perform the same clinic activities i.e. they're not a place where patients stabilize. Andrew will try to pull some example cases for us to think through the interaction between the clinical floors and the transfer algorithms. This is very little work for the data and model side. This would necessitate some sim UI i text changes.

  3. Use episode count parameters Currently, we are in danger of using medians that have been calculated from very few data points and then applying them to all patients in a setting, potentially distorting the patient outflows. We propose pulling the episode counts along with the engagement durations, which Tom can use in the model to weight the flows to better fit to the team. There are some good reasons to proceed with this change, but it would take a significant amount of extra work on both the data and the model side. The sim UI shouldn't be affected.

lzim commented 5 years ago

@TomRust @holbrooa @saveth and @saveth

For when we return to this later:

We propose defining high-symptom patients as those who are above 50% on the symptom distribution for this team.

  1. How is that defined, from their measures? If so, I’m VERY excited about this possibility.
  2. New floors and exempt PC/PCMHI - this makes a lot of sense to me. Let’s plan to meet about this after @holbrooa’s work on #615 is handed off to @saveth
  3. Yes, @TomRust and I talked about this at our meeting last week, and as a health services researcher working with a better estimate of the distribution of engagement durations makes a ton of sense to me. But, again, if this takes a fair bit of time, we do need to support a few other key priorities, do our best to come up with a cost estimate for this and then return to it.

Thanks so much All 😎

lzim commented 5 years ago

@holbrooa

@TomRust and I met today and want to return to this if your hand-off to Savet is complete, and the documention on OSF is done.

- Do we need a meeting?

  1. The definition now, is "high symptom" is high symptom based on stepped up decisions, to the particular team they selected in UI.
    • Instead, the recommendation is to change the definition of who is high- and low-symptom, by assuming that the patients are normally distributed on symptomology within the team and splitting them 50/50 into the high symptom and low symptom stocks.
    • This would reflect 50/50 at the outset of care in that team, so it just reflects the upper and lower 50% of whatever proportion of the patients in that team have historically been. It doesn't say anything about the proportion of patients with "high" or "low" symptom scores relative to the general Veteran outpatient population.
    • This seems like safer, or more defensible position than taking the limited actual symptoms score and generalizing them to the entire team.
    • It also is more defensible than our prior way of defining patients based on where they go later, because this part of the model is at the outset of treatment, and it is not reasonable to use where they end up after care in this team as our definition of their high/low distribution at the outset of care in this team.

Decision: - CHANGE 1 is APPROVED.

2. Let's review the case examples together, using the updated PC/PCMHI data guidance we received from Jodie. Cross-ref #596

Next Steps: @holbrooa

3. The recommendation is: "Pulling the episode counts along with the engagement durations, which Tom can use in the model to weight the flows."

Next Steps:

@staceypark FYI: And, we may need a meeting 😄

holbrooa commented 5 years ago

@holbrooa and @TomRust met and talked at length about these issues. We have a multi-pronged plan involving model changes and data changes. But first, some data exploration. We came up with a reasonable set of data to review, which @holbrooa will work on then @lzim can see it and make some decisions next week. I will try to have it done early in the week, to give myself some time to implement any changes resulting from that meeting.

lzim commented 5 years ago

Great! Thanks @TomRust and @holbrooa

I’m presenting at a conference next week, but ping me on GitHub and I will take a look.

Lindsey

staceypark commented 5 years ago

@holbrooa @TomRust I believe you guys were going to provide an update on potential options we could pursue to resolve this. Could you two both join the Monday Workgroups meeting to run by your options with @lzim?\

Let's update this issue with as much detail before hand so we can be efficient.

TomRust commented 5 years ago

New idea from Lindsey: Include the 1-n-dones as an outflow from "Early in Care"!!!

holbrooa commented 5 years ago

We have various options for how to deal with and how to explain the potential mismatch between the floors and the data. I'll make a separate post here.

holbrooa commented 5 years ago

Option A: Fixing the bug mentioned above and removing the 0-length engagements (and potentially moving them to a separate outflow as @lzim suggests) actually boosts the parameters above the level of the floors for the teams that @holbrooa has tested recently. So we could just make those two small data fixes and not change anything else. The removal of 0-length engagement is very easy to explain if they form a separate outflow in the case of ending. It's less clear for the step ups/downs.

So option A1 is removing the 1-n-dones and leaving it at that. A2 is implementing them as their own outflow. But what do we do with the 1-n-steps?

Option B: Do option A but also change the time to detect to allow a value of 0.1, giving us a lot more probability of our data parameters coming in above the floor. This could be explained as the mix of situation 1: "new episodes of care for existing patients," in which the time to detect often happens before our algorithm actually detects the new episode, and situation 2: new episodes for patients being referred in from elsewhere, where the detection really does have to happen in/after that first visit. If you have as much or more of situation 1 as you do of situation 2 (the data suggest this is usually the case, at least for GMH) then a median time to detect of 0 (ish) is totally reasonable.

@TomRust and @holbrooa also discussed getting rid of floors altogether and an option where teams pick their own time to detect. I think that latter case could just be option B above. We can also pull episode counts alongside the various parameters. I'm not totally sure how they'd be used.

staceypark commented 5 years ago

Engagement duration will now only include duration in episode in treatment. And no longer includes the time/work it takes for the care team to make a decision. Time to Detect will no longer influence other outflows of Engagement Duration. The two variables are completely separate of each other now. (This accounts for patients who may have Engagement duration of 0-1 weeks, or patients that start new episode from being existing patients).

@TomRust and @holbrooa will update the model and parameters respectively based on changes discussed above. @TomRust & @holbrooa will craft updated i information text for @lzim & @staceypark to review.

holbrooa commented 5 years ago

Quoth the server: Four oh four.

                                             ,::::.._
                                           ,':::::::::.
                                       _,-'`:::,::(o)::`-,.._
                                    _.', ', `:::::::::;'-..__`.
                               _.-'' ' ,' ,' ,\:::,'::-`'''
                           _.-'' , ' , ,'  ' ,' `:::/
                     _..-'' , ' , ' ,' , ,' ',' '/::
             _...:::'`-..'_, ' , ,'  , ' ,'' , ,'::|
          _`.:::::,':::::,'::`-:..'_',_'_,'..-'::,'|
  _..-:::'::,':::::::,':::,':,'::,':::,'::::::,':::;
    `':,'::::::,:,':::::::::::::::::':::,'::_:::,'/
    __..:'::,':::::::--''' `-:,':,':::'::-' ,':::/

.::::::,:::.-''--..','. ,', , ' , ,' ', ',' ,::SSt:'''' \:. . ,' ' ,',' ',' ``::.,'',',.-' \ \ \\ \-.-' .-.\\__. -.-._ `

jamesmrollins commented 5 years ago

😂

staceypark commented 5 years ago

@TomRust @holbrooa decided to leave Time to Improve as is, but we do need @TomRust to review definition for Time to Improve & Time to Detect and make sure they reflect decisions made during the meeting.

In @holbrooa 's data UI, everything to the right of the numbers are not dependent on the code and can be changed in final_datafiles before propagation. For anything to the left of the numbers, it needs to be updated in the code.

ritahitching commented 5 years ago

@holbrooa Any update on the code to the Data UI?

holbrooa commented 5 years ago

Yeah the SP stuff that we talked about is all implemented in nevermore. The descriptions aren't current anymore, though. Those can and should be changed directly in nevermore. For example, the symptom proportions are now spitting out 0.5 instead of a calculated value, but it still has the old description describing how it used to be calculated.

staceypark commented 5 years ago

@holbrooa where would we find the current descriptions so we can update? would @TomRust have them or do we need to draft them?

holbrooa commented 5 years ago

I'm talking about the descriptions in the Data UI, so you'll find them in the relevant columns of SPParams in nevermore. Those description columns are static (i.e. not coming from the code) so you can just change it right there.

holbrooa commented 5 years ago

I don't think there's a new description anywhere for you to just use. Like, the data UI is out of date in this known way, and I don't think new language has been drafted for this parameter in any crosswalk or sim UI or anything. So yes, you'll need to draft something new. I can probably call in to a meeting or something to help with that, but I'm in SLC all week next week, so it might be easier to just email me whatever you come up if you want my double-check.

lzim commented 4 years ago

Revisiting decisions made on 10/23 Plan is to follow-up week of Jan. 6, 2020

FYI: @TomRust @staceypark @jamesmrollins @holbrooa

Tom, Stacey and Lndsey met and tested and reviewed SP in Nevermore.

  1. Symptom Proportions definition: New "Team Data Table" and "i" information for Master Cross Walk and Sim UI

Symptom Proportions New/edited definition edited in "Nevermore:" "An estimate of the proportion of patients engaged in episodes of care in each setting  who have high symptoms with 50% above and 50% below the median. (pct)"

Question: Can we edit the "i" information that pops off to the side of the Team Data Table, or only individual pop-ups?

  1. Re-cap: 0 length engagement time - we will exclude zero length engagements (i.e., "one and done") as this not an episode of care. And therefore, these zeros should not be included in calculations of medians (zero-inflation). This includes engagement time before step up/down & engagement time before ending.

  2. Question about how we did or did not resolve 1-n-step and 1-n-done situations?

    • Need to ask @holbrooa about this. Can you help us remember this, Andrew?
  3. Based on record "Time to Detect" bc is 4 weeks.

  4. Could not identify "Time to Improve" related decisions.

  5. Still need to make a decision using the Episode Counts about whether and how to display.

    • Design idea modeled on Yelp or Amazon, where you see the distribution of ratings and weigh it more heavily when there are lots of raters. We want to weigh more heavily engagement durations that are based on many more episode counts.
  6. Try to schedule a meeting in January with Tom, Lindsey & James (try to touch base with Andrew before then).

Thanks!

staceypark commented 4 years ago

@jamesmrollins Could you update the 'i' pop-up for the Team Data Table in SP with the updated definition for Symptom Proportions:

  1. Symptom Proportions definition: New "Team Data Table" and "i" information for Master Cross Walk and Sim UI

Symptom Proportions New/edited definition edited in "Nevermore:" "An estimate of the proportion of patients engaged in episodes of care in each setting  who have high symptoms with 50% above and 50% below the median. (pct)"

jamesmrollins commented 4 years ago

@staceypark will you please confirm the change below?

@hirenp-waferwire - please make the change indicated below in the SP Team Data Table.

image

staceypark commented 4 years ago

Looks correct

dlkibbe commented 4 years ago

Hi @TomRust - An action item in Lucid says, "Also need to comb through an look for any mentions that states MTL units are "Patients" or "Appointments" because we also have the unit: "Episodes of Care" for SP." @branscombj and I went through the SP module and found 2 occurrences of Episodes of Care on the model diagram linking dials or rectangle measured in patients; 2 mentions of Episodes of Care in the team data table measured in patients/week or weeks, and one 1 experimental variable measured in patients/week. Screen shots below. Not sure how to best edit the see/say files given the Episodes of Care are measured mainly in patients/week or weeks. Talked with @jamesmrollins @lzim about this today and suggestion was to put the information here. Thanks. image

image

jamesmrollins commented 4 years ago

Discussed this issue with @TomRust last Friday. We reviewed the episodes count issue and determined that if less than 1% of the total population of episodes of care is returned from the team data query, than the sample size should be considered insufficient. If the sample is insufficient, then the model should read the number as a zero; thereby, shutting down the flow in question and preventing the user from making a less-than-plausible inference.

We discussed two ways to manage the logic for this:

  1. At the Team Data query level. If the query does not return a sufficient sample, it returns a zero. Need @anazariz and @anthonycpichardo to evaluate for level of effort.
  2. At the model level. If the sample size is insufficient, then the model logic will ignore the value and return a value of zero.

In either case, insufficient data will return zero, thus not putting the user in a position to judge whether or not the model extrapolation is valid. There would be no graphical warning or measurement such a value weighting similar to Yelp or Amazon style ratings necessary, since the sim would not present insufficient measurements in the form of team data in the first place.

@TomRust indicated it would take 10 hours to complete the task at the model level.

anthonycpichardo commented 4 years ago

It would about 5 hours of effort on our end due to some research that @anazariz and I would have to do prior to making the change. @jamesmrollins

lzim commented 4 years ago

@jamesmrollins and @TomRust

How would this zero be explained to the user?

@anthonycpichardo @anazariz

jamesmrollins commented 4 years ago

@lzim @TomRust @anthonycpichardo @anazariz couldn't we put an explanation in the Team Data table? This assumes they are looking at the Team Data when they download it from the portal.

Otherwise, we are back to trying to figure out how to graphically warn them in the Team Data Table in the very busy Experiments Section, which could be accomplished by:

image The yellow cells indicate numbers that do not have a sample size adequate for representation in the model.

image

staceypark commented 4 years ago

@jamesmrollins @lzim @anthonycpichardo I moved this to the current Epic, as I believe this will affect documentation for go_live.

Updates from James 2020/02/24 workgroup leads meeting:

  1. Discussed with @TomRust.
  2. Determined that if less than 1% of total episodes, measurement should be considered "0." That would effectively eliminate the given flow from the model.
  3. Can apply this rule in either the model or the team data query. Quant indicates about 5 hours to research and implement.
  4. If we remove insufficiently reliable data from the user's view then we wouldn't necessarily need a graphical moniker such as a Yelp or Amazon value weighting scheme.
  5. If implemented in model and SIM UI, 8 hours for Model Development, roughly the same for SIM UI.

ISSUE: We need to make a decision on where this will be implemented (quant, model/sim) and clarify which documents it will affect.

DECISION: We decided to resolve this issue at this Thursday's Support Workgroups Meeting.

jamesmrollins commented 4 years ago

From Support Groups Meeting 2/27

lzim commented 4 years ago

Guide language workshopped during the meeting

Our goal is to make the learners aware when low counts used for estimates. A key systems insight for facilitators is that teams will always find the highest leverage change focusing on where the patients are (i.e., stocks or flows).

Sometimes we may have estimates derived from very few observations. We decided to set infrequently observed values to zero to avoid inflation of rarer episodes of care.

Poor inferences are possible using estimates when the count is low and the duration is low. If the values are <1% of the total counts of episodes of care in the team, and they are retained in the model, it will inflate rare episodes.

Here is example competing for the same outflow of patients from PC/PCMHI PC/PCMHI to GMH

PC/PCMHI to SMH

Because the flow is based on the engagement duration (weeks), below is an example 30:1 difference in episode counts, but the difference in engagement duration is actually only 2:1 (30 weeks to 14 weeks).

Retaining this parameter to estimate these model outflow (pts/wk) over the course of a two year experiment timeline would exaggerate the number of patients who step up from PC/PCMHI to SMH from 1 to 20 and reduce the number who step up from PC/PCMHI to GMH from 30 to 10.

If we set it to zero it won't include this rare flow in the basecase. Because the outflow that is infrequently observed is set to zero, it will slightly inflate the other stock outflows, but this is better than extreme exaggeration of an uncommon episodes of care.

However, the Measurement Based Stepped Care for Suicide Prevention module includes low base rate patterns for high risk flag patients.

The outflows from the HRF Patients stock are to residential

Sim UI - Setting View [CURRENT IS]: PC/PCMHI - Residential or Unflag (2 outflows) GMH - Residential, GMH or Unflag (3 outflows) SMH - Residential, GMH or Unflag (3 outflows) - no change

Setting View [CHANGE NEEDED IS]: PC/PCMHI - Residential, GMH or Unflag (3 outflows) GMH - Residential or Unflag (2 outflows) - likely to keep their HRF flag patients SMH - Residential, GMH or Unflag (3 outflows) - no change

Inpatient was not included as an outflow from any setting view because the patient never really transfer fully to the inpatient setting, they are still typically managed by the team coordinating their treatment before the inpatient stay and after discharge from an inpatient stay.

The episode counts for HRF patients are always likely to represent a low percentage of the total episodes of care in the team.

We use the same logic for parameters that control the outflows from the HRF patients stock. These parameter estimates are set to zero when they are < 1% of the HRF episodes of care, and not when they are < 1% of the total episodes of care in the team.

jamesmrollins commented 4 years ago

Hi @TomRust, I know you are busy, so I will just put my inputs here. In order for the simulation to turn on an exclamation point, it will need a signal from the model. To that end I propose that we create a data variable structure in the example below. I count about 50 data variables (not including “adjusted”), although I don’t think they are all related to this problem.

Variable Name What it does in the model What is does in the SIM UI
Data Sum of Episodes Carries the sum of all episode counts Not used by  Sim UI
Data GMH to PC/PCMHI  Engagement Time before Step down Flag If the episodes of care associated with this Engagement Time are less than 1% of total episode counts, then = 1.  The 1 can be used to turn the value of the flow to “0.” The sim logic will read the “1” and turn the related “!” icons next to the team data table entry and the related parts of the flow diagram.

What do you think? Let me know if I can help. James

staceypark commented 4 years ago

@jamesmrollins @lzim Just touch based quickly with @TomRust who is hoping to turn to and resolve this on Monday. Tom has been made aware of the documentation dependencies that are waiting on his work.

staceypark commented 4 years ago

@TomRust has blocked out time next Monday 3/23 to check the models and the <1% threshold.

lzim commented 4 years ago

@TomRust @staceypark @jamesmrollins

The "SP floors" decision rules only refer to the basecase:

  1. Exclamation points in the team data table will indicate when a parameter is too low to be validly used.
  2. The relevant red parameter values in the model diagram will have a corresponding exclamation point
  3. You will get a pop-up message from clicking on the exclamation point in either the Team Data Table or the Model Diagram to explain what is happening.
  4. Any rate or stock that is zero is greater transparency - Because the impacts of a "exclamation point" will zero-out "downstream" flows and stocks, James will see if he can use the zero flows and zeros stocks to signal to the Sim UI that greater transparency is needed visually to alert the learner that this section of the model is "off."

For experiments you can turn these sections back "on:"

  1. The exclamation in the Team Data Table over the relevant zero would remain.
  2. Changing to a non-zero value with an experiment slider would a) remove the transparency, b) remove the exclamation points in the model diagram, and c) engage that section of the model.

"!" pop-up language: "This parameter value is < 2% of total episodes of care, which is an unreliable estimate. In the basecase, the parameter reads "0" to disengage this model flow. However, a non-zero value can be used in an experiment."

lzim commented 4 years ago

@staceypark Can you please confirm with @anthonycpichardo & @jamesmrollins about using the correct Team 4 data. Thanks!

branscombj commented 4 years ago

@staceypark @lijenn @jamesmrollins @dlkibbe and I are not sure if we have a documentation updates task tied to this and, if so, whether that has to wait for this to be implemented or we (you:)) know enough now for those updates to be made? Thanks!

lzim commented 4 years ago

@TomRust @jamesmrollins

@branscombj @dlkibbe I think wait on documentation until we get an update from Tom

*Also, I edited the title, milestone and epic to reflect may_epic

lzim commented 4 years ago

@TomRust I just wanted to quickly ping to see if you are able to post the model, and hypotheses for the users you want to review it. I know your attention will be focused elsewhere (or maybe already is?) - let us know. Thanks!

TomRust commented 4 years ago

Hi all! @lzim @jamesmrollins @branscombj @dlkibbe @staceypark

Just trying to close this issue before I’m in newborn-land. I’d like feedback on a prototype from both James and some facilitators that addresses the three issues raised at the top of this issue thread. Unfortunately, the prototype is just a Vensim model, so I guess the only thing to for facilitators to test is the description of the use case? I hope that works! I’d love for Debbie and Jane to see how this change to SP fits into the facilitation of Session 5, and James to check out all the related SimUI changes.

I've split the hypotheses to be tested into three comments, as there were three "issues" raised that started this thread in the first place. I assume it will be easier to track responses when they are separated.

--TOM

TomRust commented 4 years ago

Address part 1 (from top of issue thread): The symptom % is now hard-coded to be a 50/50 split, which means that we're defining "high symptom" as the any patient that has greater than average symptoms, and "low symptom" as less than average symptoms. Up to now, we've been defining high and low symptom by where a patient transfers to (e.g., a patient who steps up is labeled "high symptom"), and then the model takes these "where they went" data and calculates a their initial symptom state based on all the other parameters...which is very hard to explain and added seconds to the sim run time.

Use-case to test for SP sessions for @branscombj and @dlkibbe : End user – In session 3, we produce the team data table, and are told that the model includes an a few important dynamic that we don't have good data for (yet) in VHA: 1) patients get better when they're in treatment, 2) not all patients have the same severity of symptoms . . . and others that i don't remember now :) . When explaining how the model captures patients coming to our setting with a distribution of symptoms, the model simplifies this to be half of patients start out "high" symptom (i.e., with more of / more severe than average symptoms) and the other half start out "low" symptom.

Sim Dev @jamesmrollins – Now that we've changed the underlying definition, the symptoms % is no longer based on team data.

  1. Can we remove symptom proportions from the team data table in the Sim UI?
TomRust commented 4 years ago

Address issues #3: I included a “parameter scrub” logic that uses a "threshold for inclusion" based on whether or not a parameter is based on >2% of either total HRF or no-HRF care episodes (across all the clinics the team selected).

Use case to test by facilitators @dlkibbe @branscombj -- End-user: I’m in session 5, reviewing how all my patients’ data are used to customize the sim. My team data is has some “wacky” values that I don’t trust/don’t mesh with my experience. But, looking at the “episode count column” in the model parameters file shows me that these numbers are based on some “one-off” incidents, and thus should probably be ignored as statistically insignificant. My facilitator explains that model parameters based on a tiny fraction of the care we deliver (less than 2% of the total episodes for either patients with or without a High Risk Flag) are excluded from the simulation. This allows me to focus my learning on what happens most of the time, to find change ideas that will improve care for the most patients. They reassure me that we can always add any of these values back into the simulation as an experiment, if we want.

Hypotheses to test:

a. End-users will no longer be distracted by “invalid” model parameters based on a small number of care episodes b. Learning will go faster, as the simulation behavior will no longer be skewed by these “untrustworthy” parameter values c. Easier for users and facilitators to focus on the most important flows in the model, as the “less significant” flows will be turned off by leaving out these parameter values, and the whole SP model will become simpler

Questions for the @jamesmrollins : The Vensim now has all these additional variables and equations that can be used to modify the graphics in the SimUI. Last time we talked, we came up with the idea that if a parameter fell below our “threshold for inclusion,” then that parameter would be literally flagged (with a red exclamation point, right?) in the team data table, these exclamation points would have a pop-up with explanation text, and the related model structure (the flow that parameter governed) would be “grayed out” or made noticeably more transparent. (see note from March20 in this issue)

  1. Can you upload a mock-up?
  2. I've given all the variables in Vensim that would trigger these exclamation points to appear the prefix "flag...," where a zero means the parameter falls below our threshold for inclusion and thus the exclamation point should show up in the Data Table. Does those variables work?
  3. Graying out the rate in the model diagram related to any of these excluded parameters is straightforward, but graying out some stocks will need a bit of logic, as some stocks have more than one inflow. Any stock should only be “grayed out” if all its inflows are zero. Also, if a stock is grayed out, then all its outflows should also be “grayed out.”

Three questions for everyone on the thread:

  1. Do we want different thresholds for HRF and no-HRF data? The numbers of HRF episode counts are so small that it is very unlike to ever have <2%. Maybe 5% or 10% would make more sense for HRF patients.
  2. Do we want the threshold to be 2%? I've only been able to see one example SP model parameter excel file, so I'm not sure if that is low enough to really cut out all the "noise."
  3. Do we want the end users to be able to adjust this threshold? My sense is "no," but that tickles my "participatory" sense.

I dropped the new version of SP with these changes here: teampsd/model_workgroup/models/mtl_1.8_models/

TomRust commented 4 years ago

Finally, issue part 2, about modifying team data "floors" and Time to Detect:

I haven't made any model changes about this part, but could very easily. I think we should get rid of the floors, but wanted to run it by you all first. Here is a discussion of the details:

1) Do we keep “floors” for Time to Improve and Engagement Duration before Step Up/Down?

Facilitator changes: We remove references to data "floors" in guides for sessions 3 and 5. Sim UI changes: We remove references to data "floors" in the i text in the Team Data Table.

2) Do we keep base case Time to Detect set at 24 weeks for PC, 12 for GMH, and 4 for SMH?

jamesmrollins commented 4 years ago

@TomRust @dlkibbe @branscombj please see the mock ups and responses to Tom's questions below:

Sim Dev @jamesmrollins – Now that we've changed the underlying definition, the symptoms % is no longer based on team data.

  1. Can we remove symptom proportions from the team data table in the Sim UI?

ANSWER: Yes. The illustration below shows that line removed.

Questions for the @jamesmrollins : The Vensim now has all these additional variables and equations that can be used to modify the graphics in the SimUI. Last time we talked, we came up with the idea that if a parameter fell below our “threshold for inclusion,” then that parameter would be literally flagged (with a red exclamation point, right?) in the team data table, these exclamation points would have a pop-up with explanation text, and the related model structure (the flow that parameter governed) would be “grayed out” or made noticeably more transparent. (see note from March20 in this issue)

  1. Can you upload a mock-up?
  2. I've given all the variables in Vensim that would trigger these exclamation points to appear the prefix "flag...," where a zero means the parameter falls below our threshold for inclusion and thus the exclamation point should show up in the Data Table.  Does those variables work?
  3. Graying out the rate in the model diagram related to any of these excluded parameters is straightforward, but graying out some stocks will need a bit of logic, as some stocks have more than one inflow. Any stock should only be “grayed out” if all its inflows are zero. Also, if a stock is grayed out, then all its outflows should also be “grayed out.”

ANSWERS: Regarding #1 - Mock-ups are shown below. Regarding #2 - An exclamation point inside a red circle icon is shown by all flagged values. When the user clicks the red icon, a pop up with explanation text is presented. Regarding #3 - I don't think we can support graying out sections of the diagrams. However, we can put red icons near the affected variables. We may need to expand the explanation in the Pop Up that explains that since the threshold is not met, the variable is returning a "0" value. Also note how many variables are affected by SMH values that fall below threshold. Lights up much of the Team Data Table.

Team Data Table Mock-up

image

Diagram Mock-up

image

"!" Icon Pop up

image

branscombj commented 4 years ago

We may need to expand the explanation in the Pop Up that explains that since the threshold is not met, the variable is returning a "0" value.

Am I understanding this correctly: The values with exclamation points would always be zeroes?

jamesmrollins commented 4 years ago

Hi @branscombj , yes, correct. If the Team Data value does not meet the 2% threshold, a zero gets passed from the variable to all the other variables with which it is connected. Therefore, if the variable is multiplied in the downstream relationship, the product will be zero. However, if associated with a user-defined variable (slider), then the slider value would be reflected in the calculation.

In the illustration above, the red slider "SMH Recommended New Episodes of Care Rate," would be initially zero because the corresponding value in the Team Data Table (1.02) is below the 2% threshold. This should indicate to the user that the value in the Team Data Table is statistically unreliable; therefore, not used (set to zero). However, the value can be experimentally manipulated using the slider to a desired value.

I am available - please let me know if I can be of further assistance.

jamesmrollins commented 4 years ago

Hi @lijenn @dlkibbe @branscombj .

Vensim Model Variables for SP Floors

image

Updated Team Data Table with Added "!" Icons in the "Time to Improve" row.

image

jamesmrollins commented 4 years ago

Hi @lijenn @branscombj @dlkibbe . The updated Sim UI with floors and episode count flags is available Tuesday in test. I want today to take a look, so don't look until tomorrow. I need to update the "i" information and stuff. We will probably need to look through the See and Say guides. Also take a look at the explanatory language for the "i" information. I will double check your TEST logins in Epicenter (it's been awhile) and make sure you have current logins.

jamesmrollins commented 4 years ago

Hi @hirenp-waferwire. Please find the following changes/corrections:

1. Please add the "!" icons to the main diagram.

image

2. Please correct the variable mapping.

image

jamesmrollins commented 4 years ago

@branscombj @lijenn @dlkibbe

jamesmrollins commented 4 years ago

Hi @lijenn, @branscombj and @dlkibbe.

Bottom line up front (BLUF): I have completed the design of the find-and-replace, spell checker, link checker and the markdown style linter. The instructions for their use is in the Team PSD SOP. I think we wanted to use this issue as a test case for the actions. If you would like, I can be available to provide you with support during your next workgroup meeting or whenever.

Below are a few notes for your consideration.

lzim commented 3 years ago

@jamesmrollins and @lijenn

What is the latest on this? Thanks!