-
Notifications
You must be signed in to change notification settings - Fork 24
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
7 changed files
with
90 additions
and
61 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,95 +1,59 @@ | ||
| # TODOs | ||
|
|
||
| ## Business logic, general features | ||
| - [x] Issues guide for the GitHub (OS, content of the config, project tree, enable verbose) | ||
| - [ ] GitHub CHANGELOG - separate New, Fixed, Changed into paragraphs | ||
| - [ ] Middleware support (FreeRTOS, etc.) | ||
| - [ ] Arduino framework support (needs research to check if it is possible) | ||
| - [ ] Create VSCode plugin | ||
| - [ ] UML diagrams (core, GUI back- and front-ends, thread flows, events, etc.) | ||
| - [ ] CI is possible (Arch's AUR has the STM32CubeMX package, also there is a direct link). Deploy Docker one in Azure Pipelines, basic at Travis CI | ||
| - [x] We can hide almost all logging setup behind the scene. Think of it as of a default implementation that can be changed though. Also, can make a `setup_logging()` function | ||
|
|
||
| ## GUI version | ||
| - [ ] Obtain boards on demand (not at the startup) | ||
| - [x] Expose version to the About dialog | ||
| - [x] Handle a theoretical initialization error (when boards are receiving) | ||
| - [x] Maybe `data()` `QAbstractListModel` method can be used instead of custom `get()` (no, at least without any tweaking (`QModelIndex` vs `int`)) | ||
| - [ ] Can probably detect Ctrl and Shift clicks without moving the mouse first | ||
| - [x] Notify the user that the 'board' parameter is empty | ||
| - [ ] Mac: sometimes auto turned off shift highlighting after action (hide-restore helps) | ||
| - [ ] Some visual flaws when the window have got resized (e.g. 'Add' button position doesn't change until the list gets focus, 'Log' area crawls onto the status bar) | ||
| - [x] Gray out "stage" line in all projects except current | ||
| - [ ] Tests (research approaches and patterns) | ||
| - [ ] Remade the list item to use States, too. Probably, such properties need to be implemented: | ||
| ``` | ||
| state: { | ||
| loaded, | ||
|
|
||
| visitedAfterInstantiating, | ||
|
|
||
| actionRunning, | ||
| lastActionStatus, | ||
| visitedAfterAction, | ||
| ... | ||
| } | ||
| ``` | ||
| - [x] Test performance with a large number of projects in the model. First test was made: | ||
| 1. Some projects occasionally change `initLoading` by itself (probably Loader unloads the content) (hence cannot click on them, busy indicator appearing) | ||
|
|
||
| Note: Delegates are instantiated as needed and may be destroyed at any time. They are parented to ListView's contentItem, not to the view itself. State should never be stored in a delegate. | ||
|
|
||
| Use `id()` in `setInitInfo()`. Or do not use ListView at all (replace by Repeater, for example) as it can reset our "notifications" when reloading | ||
| 2. Some projects show OK even after its deletion (only the app restart helps) | ||
| - [ ] Test with different timings | ||
| - [ ] Divide on multiple modules (both Python and QML) | ||
| - [ ] Implement other methods for Qt abstract models | ||
| - [ ] Warning on 'Clean' action (maybe the window with a checkbox "Do not ask in the future" (QSettings parameter)) | ||
| - [x] 2 types of logging formatters for 2 verbosity levels | ||
| - [x] `TypeError: Cannot read property 'actionRunning' of null` (deconstruction order) (on project deletion only) | ||
| - [ ] QML logging - pass to Python' `logging` and establish a similar format. Distinguish between `console.log()`, `console.error()` and so on | ||
| - [ ] Lost log box autoscroll when manually scrolling between the actions | ||
| - [ ] Crash on shutdown in Win and Linux (errors such as `[QML] CRITICAL QThread: Destroyed while thread is still running Process finished with exit code 1073741845`) | ||
| - [x] Start with a folder opened if it was provided on CLI (for example, `stm32pio_gui .`) | ||
| - [ ] Linux: | ||
| - Not a monospaced font in the log area | ||
| - [ ] Linux: Not a monospaced font in the log area | ||
| - [ ] Temporarily pin projects with currently running actions to the top (and stay there on scrolling). See QML Package type | ||
| - [x] Icons instead of text for "Clean", "Open editor" | ||
|
|
||
| ## Core library | ||
| - [x] https://github.com/ussserrr/stm32pio/issues/13 | ||
| - [ ] when updating the project (`generate` command), check for boards match | ||
| - [x] Remove casts to string where we can use path-like objects (related to Python version as new ones receive path-like objects arguments while old ones aren't) | ||
| - [ ] Remove casts to string where we can use path-like objects (related to a Python version as new ones receives path-like objects arguments while old ones aren't) | ||
| - [ ] We look for some snippets of strings in logs and output for the testing code but we hard-code them and this is not good, probably (e.g. 'DEBUG') | ||
| - [ ] Store an initial content of the folder in .ini config and ignore it on clean-up process. Allow the user to modify such list (i.e. list of exclusion) in the config file. Mb some integration with `.gitignore` | ||
| - [ ] at some point check for all tools (CubeMX, ...) to be present in the system (both CLI and GUI) (global `--check` command (as `--version`), also before execution of the full cycle (no sense to start if some tool doesn't exist)) | ||
| - [ ] generate code docs (help user to understand an internal mechanics, e.g. for embedding). Can be uploaded to the GitHub Wiki | ||
| - [ ] colored logs, maybe (breaks zero-dependency principle) | ||
| - [x] check logging work when embed stm32pio lib in a third-party stuff (no logging setup at all) | ||
| - [x] merge subprocess pipes to one where suitable (i.e. `stdout` and `stderr`) | ||
| - [x] redirect subprocess pipes to `DEVNULL` where suitable to suppress output (tests) | ||
| - [ ] maybe migrate to async/await approach in the future (return some kind of a "remote controller" to control the running action) | ||
| - [ ] `__init__`' `parameters` dict argument schema (Python 3.8 feature). | ||
| - [x] See https://docs.python.org/3/howto/logging-cookbook.html#context-info to maybe remade current logging schema (current is, perhaps, a cause of the strange error while testing (in the logging thread), also it modifies a global settings (log message factory)) | ||
| - [x] Test preserving user files and folders on regeneration and mb other operations | ||
| - [x] Move special formatters inside the library. It is an implementation detail actually that we use subprocesses and so on | ||
| - [ ] Mb store the last occurred exception traceback in .ini file and show on some CLI command (so we don't necessarily need to turn on the verbose mode and repeat this action). And, in general, we should show the error reason right off | ||
| - [x] 'verbose' and 'non-verbose' tests as `subTest` (also `should_log_error_...`) | ||
| - [ ] the lib sometimes raising, sometimes returning the code and it is not consistent. While the reasons behind such behavior are clear, would be great to always return a result code and raise the exceptions in the outer scope, if there is need to | ||
| - [ ] check board (no sense to go further on 'new' if the board in config.ini is not correct) | ||
| - [x] check if `platformio.ini` config will be successfully parsed when there are interpolation and/or empty parameters | ||
| - [x] check if `.ioc` file is a text file on project initialization. Let `_find_ioc_file()` method to use explicitly provided file (useful for GUI). Maybe let user specify it via CLI | ||
| - [x] mb add CLI command for starting the GUI version (for example, `stm32pio --gui`) | ||
| - [ ] test using virtualenv | ||
| - [ ] test for different `.ioc` files (i.e. F0, F1, F4 and so on) as it is not the same actually | ||
| - [ ] mb allow to use an arbitrary strings (arrays of str) to specify tools commands in stm32pio.ini (shell=True or a list of args (split a string)) | ||
| - [ ] cache boards for a small interval of time | ||
| - [x] use warnings.warn() (https://docs.python.org/3/howto/logging.html#logging-basic-tutorial) | ||
| - [ ] count another '-v' as '-v' for PlatformIO calls (slider in GUI settings window) | ||
| - [x] move GUI-related stuff from the `util.py` | ||
| - [x] typing (`Mapping` instead of `dict` and so on) | ||
| - [ ] check imports from 3rd-party code when the stm32pio installed from PyPI | ||
| - [x] Update embedding example (and maybe move to the repo itself) | ||
| - [ ] Project' name (path) can be reused so cannot be used as a unique identifier but so is id(self)? Probably it is better to use a path (human-readable) | ||
| - [x] Inform user that given parameters have overridden the config | ||
| - [ ] Analyze `.ioc` file for the wrong framework/parameters |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.