tickets/SP-3045: Remove unused functions to reduce/remove collect dependencies in plot

[ehneilsen]

No description provided.

[rhiannonlynne]

You know better than I where these were being used, but it might be good to comment which functions were removed and what people could use alternatively (in a comment in the PR perhaps).

[ehneilsen]

The removed functions are:

• create_survey_reward_plot
• create_visit_explorer

To my knowledge, these are currently unused anywhere. In both cases, the functions are combinations of collect/compute/combine steps implemented in other functions. When the reward plot made by create_survey_reward_plot is made currently (in the prenight briefing notebook), the notebook itself calls the various underlying functions directly, not this driver. The create_visit_explorer function was only ever an exploration tool, and is better called through underlying functions as well.
Note that these functions mildly violate the high-level architecture of schedview, and this violation was the cause of the issues encountered that led to the jira issue being filed. If similar functionality is desired in the future, it should be implemented outside of the schedview.plot sub-module.