CrayLabs / SmartDashboard

SmartSim Dashboard Package.
BSD 2-Clause "Simplified" License
6 stars 2 forks source link

Add __main__ for a single point of entry to build the dashboard #29

Closed AlyssaCote closed 11 months ago

AlyssaCote commented 11 months ago

There was an issue with the smart cli where running smart dashboard with arguments wasn't calling the correct cli function in the dashboard repo in order to build itself, however, smart-dash was working perfectly fine. It turns out that calling smart dashboard was invoking the execute function and entirely bypassing the cli function which actually runs streamlit with the correct arguments attached.

Note: If this is approved, there will need to be a slight change made to the SmartSim repo as well. In plugin.py when calling dynamic_execute, the first parameter should be changed to "smartdashboard"

Another note: credit goes to Toast