Better formatting of analysis page

[mccalluc]

• Fix Better formatting of demo help on analysis page #150
• Fix Only show warning about fake data in graph once, not for every column #138
• Builds on Move reactive values up to top level #142: Merge that first.

[ekraffmiller]

This looks good and the UI has the changes described. I noticed that output and session aren't used in analysis_server(), can they be removed?

[pdurbin]

Please resolve the merge conflicts.

[mccalluc]

@ekraffmiller

I noticed that output and session aren't used in analysis_server(), can they be removed?

not really: These aren't functions I call, instead, it's an interface defined by Shiny that we need to satisfy. If I delete them in dataset_panel I get this error:

TypeError: dataset_server() got multiple values for argument 'csv_path'

I think since it's missing expected positional parameters, it's grabbing kwargs, and things get confused.

Will resolve conflicts and ask for re-review.

[ekraffmiller]

looks good, approved