spyder-ide / ux-improvements

Discussion about UX improvements for Spyder 5 and beyond
4 stars 2 forks source link

Profiler issues #20

Open juanis2112 opened 3 years ago

juanis2112 commented 3 years ago

PREFERENCES:

impact27 commented 1 year ago
  • [ ] Output should be available to store results as plain text

Profiling results can be quite complex but we could have cProfile save the standard output of the command. This would however lead to loosing most data. So the options should be: save the profiling as a binary file or save a summary as a text file. I am not sure if just saving a summary is that useful?

* [ ]  Dropdown combobox should store the results of the past profilings and not only the “names” of the files.

* [ ]  Add “history results” option to hamburger menu to match code analysis

* [ ]  History option will trigger preferences

* [ ]  Add option to change number of history results shown

I am not against adding a history to the plugin but I would rather have a save / load function.

* [ ]  Switching tabs in editor Should update the path in the combobox

* [ ]  Remove the browse file icon

* [ ]  If you close the file, the profiling will be closed (BUT STORED IN HISTORY) and switched to the current file (load the profiling if it was ran before, load the “empty state pane” in case the profiling was not ran before for that)

* [ ]  When a profiling result is loaded from the combobox and the file is not open, this will open the file automatically.

* [ ]  When loading previous results open the file corresponding to the analysis. (Show error message if the file was deleted)

In https://github.com/spyder-ide/spyder/pull/15372 I link the profiler to the console, not the file.

* [ ]  Feedback for empty pane

Ok

* [ ]  Clearing the comparison results should not hide the “diff” columns

I think the "diff" column should be there only if there is a comparison, otherwise it clutters the API

* [ ]  Rearranging the toolbar and see where to fit the combobox path

Ok

* [ ]  Remove results section

Ok