Handle unscheduled visits systematically

[rtbailey]

Description

Depending on how unscheduled visits are displayed on the x-axis, the chart can have some unintended behavior.

(See related issue for Outlier Explorer: RhoInc/safety-outlier-explorer#68)

If visits are displayed purely chronologically, the challenge is that you end up creating a new event type that shows up in the middle of the data, but for only a small number of data points. See example of visits 2(UN) and 5(UN) below:

Our primary medical monitor user has proposed that to fix this, we sort all Unscheduled visits to the far right end of the chart, where they are still visible, but they don't interfere with seeing the data flow through expected visits.

Proposed solutions:

1. Sort all unscheduled visits to the far right end of the chart, where they are still visible but they don't interfere with seeing the data flow through expected visits.
2. Retain visits in numerical order, but create a toggle that lets the user "Hide unscheduled visits"

Test notes

Test out the new "Visits without data" and "Unscheduled visits" controls in CAT with the safetyData/ADBDS.csv data file (not safetyData-queries/ADBDS.csv).

Confirm that:

• Unscheduled Visits and Visits with no data are shown/hidden when the toggle changes.
• Change measure to "Alkaline Phosphatase (ALP)" and confirm that visits with no data are shown/hidden when the toggle changes.
• Test with the above with groups active (settings: {groups:["RACE"]})
• Unscheduled visits are shown on initial load with {"unscheduled_visits":true}
• Visits with no data are shown on initial load with {"visits_without_data": true}
• Only the visits are hidden/shown when explicitly specified using {unscheduled_visit_values:["Unscheduled 2.1","Screening"]}

[jwildfire]

Test Notes: Test out the new "Visits without data" and "Unscheduled visits" controls in CAT with the safetyData/ADBDS.csv data file (not safetyData-queries/ADBDS.csv).

Confirm that:

• Unscheduled Visits and Visits with no data are shown/hidden when the toggle changes.
• Change measure to "Alkaline Phosphatase (ALP)" and confirm that visits with no data are shown/hidden when the toggle changes.
• Test with the above with groups active (settings: {groups:["RACE"]})
• Unscheduled visits are shown on initial load with {"unscheduled_visits":true}
• Visits with no data are shown on initial load with {"visits_without_data": false}
• Only the visits are hidden/shown when explicitly specified using {unscheduled_visit_values:["Unscheduled 2.1","Screening"]}