Open ljmoody opened 4 months ago
fyi @jamesmrollins @lijenn @tejaspesquare
Numbers populated correlate with what I see in the Play scenarios so I believe the flow diagram is behaving as expected.
[x] 1. Waiting to Start stock label rename
[x] 2. Patients in CC
@jamesmrollins:
@jamesmrollins - please provide guidance for the following:
Which label is correct for the Patients flow diagrams for both CC & MM?
Which label is correct for the Patients flow diagrams for both CC & MM?
fyi @jamesmrollins @lijenn @tejaspesquare
[x] 1. Numbers populated in the stocks do not correlate between the flow diagrams and what's seen in simulation Play instance. Is there a reason for this discrepancy--are each of the medication-specific services added together--or is a calculation incorrect?:
[ ] 2. Suggest revising Completing Rate dial to show 2 decimal places:
[x] 3. Change capitalization, 2 locations:
[x] 4. Change stock units of measurement labels, 2 locations:
Hi @ljmoody
Feedback for CC Flow Diagram:
CC -Modifications have been done and are now accessible on the testing environment for validation.
Hello @ljmoody
MM-Modifications have been done and are now accessible on the testing environment for validation.
Feedback for MM Flow Diagram:
- [x] 1. Numbers populated in the stocks do not correlate between the flow diagrams and what's seen in simulation Play instance. Is there a reason for this discrepancy--are each of the medication-specific services added together--or is a calculation incorrect?:
- [ ] 2. Suggest revising Completing Rate dial to show 2 decimal places:
This is not completed yet. We will allocate some time to investigate further.
[x] 3. Change capitalization, 2 locations:
[x] 4. Change stock units of measurement labels, 2 locations:
@tejaspesquare, please incorporate these additional changes:
[x] Flow pop-up: Change highlighted text to "Ending Rate"
[ ] Main Sim UI page for CC: Change both highlighted areas to be "Starting Rate"
[x] Flow pop-up: Change highlighted text to "Ending Rate"
[x] Main Sim UI page for MM: Change highlighted text to "Starting Rate"
fyi: @tejaspesquare
Hello @ljmoody
Changes are available for MM on TEST.
CC:
Flow pop-up: Change highlighted text to "Ending Rate" - Will release along with second point
"Starting Rate" field info is not reflected on simulator page hence we have not moved changes to TEST. We will release once it is resolved.
MM:
[x] Flow pop-up: Change highlighted text to "Ending Rate"
[x] Main Sim UI page for MM: Change highlighted text to "Starting Rate"
@tejaspesquare,
[ ] re: CC "Starting Rate" - I forgot to include a screenshot in my comment yesterday (7/11) but have updated it above. It's strange we don't see the same thing in CC's simulator page. This is what I see; please change both to "Starting Rate":
[x] Noticed a misspelling when looking at the CC Flow. Please change to "Appointments":
[x] Please correct spelling on this confirmation message (discovered when I added a data file to test with) to "uploaded":
Hello @ljmoody
Changes are available on TEST.
CC:
Flow pop-up: Change highlighted text to "Ending Rate" - This is moved to TEST.
"Starting Rate" field info is not reflected on simulator page hence we have not moved changes to TEST. We will release once it is resolved. !
It is required to take help from James in order to replace 'Start Rate' with "Starting Rate". He needs to update file CC_PROD.vmfx file.
*This 'name' & 'comment' (Highlighted) shall replace with "Starting Rate" & related description. Just FYI
Hello @ljmoody,
Kindly verify on TEST.
- [x] Noticed a misspelling when looking at the CC Flow. Please change to "Appointments":
- [x] apply also to MM Flow diagram
- [x] Please correct spelling on this confirmation message (discovered when I added a data file to test with) to "uploaded":
Dynamic Decision Dashboard->Team Data
Dashboard ->Settings ->Data
FYI: @tejaspesquare @ljmoody
@tejaspesquare, flows are not available in TEST instance. Please let me know when they are. Also,
@ljmoody
shall we replace 'Starting Rate' with 'Start Rate' across all 3 models MM, CC & PSY?
Yes please, @tejaspesquare w.r.t 👇
shall we replace 'Starting Rate' with 'Start Rate' across all 3 models MM, CC & PSY?
Hi @ljmoody , @matomasz
Our analysis indicates that the Starting Rate requires replacement in both the MM and Psy .vmfx files.
Discussed in DEV Standup 8/1/2024 @matthewtomo @tejaspesquare
fyi: @jamesmrollins @ljmoody
Discussed during Praxis Meeting @matthewtomo @jamesmrollins @lijenn @ljmoody
Discussed SDM-DOC CI/CD meeting 8/6/2024 @matthewtomo @jamesmrollins
fyi: @lijenn @ljmoody
Hi @matomasz
- @tejaspesquare once the files are pushed and the SDM-DOC .html file is generated, could you update it on the DEV branch of jamesmrollins/MTL-DEMO manually, since the GitHub Action is not currently working? (2024_07: BUG: Resolve SDM-DOC CI/CD action problems #3193)
DEV branch of jamesmrollins/MTL-DEMO is updated with latest files (MM & PSY models). Please be noted this was AUTO updated.
Discussed DEV standup 8/9/2024 @matthewtomo @tejaspesquare
Hello @matomasz ,
PSY & MM are up to date in Dev. Do we need to move to Prod?
PSY
MM
Hello @matomasz ,
- @jamesmrollins is currently making the changes in the MM and PSY models and will then push them to the DEV branch of lzim/mtl
We will be able to proceed for testing once vensim error is resolved.
Thank You!
Discussed DEV Standup 8/12/24 @matthewtomo @tejaspesquare
Discussed Dev Standup8/14/2024 @matthewtomo @tejaspesquare
@tejaspesquare The Agg model file is now correctly updated in the DEV environment.
week 3 2024_08 Praxis Meeting @matthewtomo @jamesmrollins @ljmoody
CC Starting Rate -> Start Rate
MM AUD Starting Rate -> AUD Start Rate DEP Starting Rate -> DEP Start Rate OUD Starting Rate -> OUD Start Rate OtherNeeds Starting Rate -> OtherNeeds Start Rate
Hello @matomasz
@tejaspesquare The Agg model file is now correctly updated in the DEV environment.
It appears that there are still some errors occurring when attempting to access the model file.
Discussed Dev Standup wk4 2024_08 @matthewtomo @tejaspesquare
@tejaspesquare
Hi Matthew,
- This issue ("VensimLibraryException: Vensim get value(INITIAL TIME) failed") should now be resolved for Agg model file in DEV
Now this is functioning properly.
Discussed Dev Standup wk_1 2024_9 @matthewtomo @tejaspesquare
Hi Matthew,
- Some of the dials are not summing correctly for the MM Flow Diagram.
This has been rectified & can be verified on Dev instance.
fyi @matthewtomo
Tested using VA laptop, Chrome browser, TEST instance.
Numbers populated correlate with what I see in the Play scenarios so I believe the flow diagram is behaving as expected.
[x] Verified all stock and dial labels
[x] Verified all "i" icon descriptions
[x] Verified inclusion of Stock/State Indicator key
[ ] Change to Completing Rate pop-up content:
[ ] Change to Referral Rate pop-up:
[ ] Feedback on flow lines/design:
[ ] Stock math isn't adding up:
fyi @matthewtomo
Tested using VA laptop, Chrome browser, TEST instance.
Numbers populated correlate with what I see in the Play scenarios and numbers in dashboard add up to what's seen on Flow diagram, so I believe the flow diagram is behaving as expected.
[x] Verified all stock and dial labels
[x] Verified all "i" icon descriptions
[x] Verified inclusion of Stock/State Indicator key
[ ] Feedback on MM Flow:
[ ] Change to Booking Rate pop-up:
[ ] Change to Completing Rate pop-up:
Hello @ljmoody ,
9/26/24 testing results for CC Flow
- [ ] Change to Completing Rate pop-up content:
- [ ] Change to Referral Rate pop-up:
We are not equipped to manage this matter. @matomasz can address this modification via directly updating the .vmfx file.
- [ ] Feedback on flow lines/design:
@tejaspesquare has provided information to the designer. I will share updates here as soon as I receive it.
- [ ] Stock math isn't adding up:
@ljmoody Could you please provide the model data file necessary to replicate the results you mentioned in your above findings.
FYI @matthewtomo
Discussed Friday Dev Standup wk_4 2024_09 @matthewtomo @tejaspesquare
@matthewtomo and @tejaspesquare, yes, I'll send the data file via Praxis Teams. FYI: I used several data files while testing, but will send the specific one used when testing and stock #s didn't add up.
- [ ] Stock math isn't adding up:
@ljmoody Could you please provide the model data file necessary to replicate the results you mentioned in your above findings.
Hello @ljmoody , @matomasz
9/26/24 testing results for CC Flow
- [ ] Stock math isn't adding up:
This has been resolved now & available to test on DEV instance.
wk1 2024_10 Ad Hoc meeting @matthewtomo @tejaspesquare
Discussed Friday Dev Standup wk_1 2024_10 @matthewtomo @tejaspesquare
Discussed Monday Dev Standup wk_2 2024_10 @matthewtomo @tejaspesquare
Discussed Friday Dev Standup wk2 2024_10 @matthewtomo @tejaspesquare
fyi: @ljmoody
Discussed Friday Dev Standup wk3 2024_10 @matthewtomo @tejaspesquare
fyi: @jamesmrollins
Discussed Dev Standup wk4 2024_10 @matthewtomo @tejaspesquare
Hi @matthewtomo,
- [ ] @tejaspesquare will communicate with Designer and/or Dev in order to get an estimate for fixing the flow arm positioning (removing the gap in-between flow arms) along with the flow arm label positioning.
- POC will be required to estimate time required for implementation.
Kindly review the recent advancements that have enhanced the positioning of labels and addressed the gap between the flow arms. Additionally, this update will manage the behavior when an image is truncated during the scaling process.
At moment this solution works on Team care (Appointments) only on Dev instance. Once you give confirmation, we will be able to proceed further.
Discussed Friday Dev Standup wk4 2024_10 @matthewtomo @tejaspesquare
The flow arms of the same categories now line up together correctly
[x] We need to remove the dashes from the labels: Example "PSY 42.50 appts/wk"
[ ] We need to ensure all labels use two decimal places
@tejaspesquare, is it possible to lessen the gaps between some of the labels?
[ ] @matthewtomo will send @tejaspesquare a model parameters file which has a large focus on one flow arm (compared to the others) for the Team Care - Appointments section.
To confirm with @lzim:
Are the gaps between some of the flow arms acceptable?
[x] @matthewtomo will test this feature further today in order to provide more in depth feedback and update the card here
Discussed Monday Dev Standup wk1 2024_11 @matthewtomo @tejaspesquare
Discussed Friday Dev Standup wk1 2024_11 @matthewtomo @tejaspesquare
We still need to develop
Other topics for @matthewtomo to discuss with @lzim:
All TEST cards for Flow:
3199 CC & MM
3072 PSY
3204 Team Care (AGG)
3205 Team Flow (SP)
AC:
[ ] all feedback to DEV is addressed and/or incorporated
~- [ ] When the user clicks on the "Flow" icon from the Dynamic Decision Dashboard, the user is presented with a new browser tab with the diagram depicted in the wireframe.~