opensatkit / cfsat Goto Github PK

The directory listings in File Browser are only updated if an action is taken from the File Browser. For example if a file is deleted then the directory listing is refreshed If a directory's content is changed from a different source such as an onboard app creating files, there's no way to refresh a listing and see the directory changes.

There are multiple cases when a dialog box shows a path that has one or more '../' in the path. Most of these are the result of a relative path specified in the cfsat.ini relative path definition being used in a join() function. Any path that will be displayed should be cleaned up.

The create app tools uses the os.mkdirs() utility which raises an exception if the destination directory already exists. A general error exception is being caught and it needs to be more specific so a 'dir/file already exists error' is not reported.

cFSAT and FileBrowser do not shutdown all of their threads properly. May be other files as well when run as main. After shutdown the GUI a control-C is required to terminate all of the threads.

When an app that has a JSON table is first initialized (see osk_c_demo) its initial status shows last table action status as invalid even though the initial table was preoperly loaded.

PySimpleGUI doesn't allow sharing layouts. The tutorial refresh recreates the main window it violates the single use layout rule. Looks like all components of the layout need to be recreated.

Tutorials are designed to provide instructions to users for how to perform tasks using cFSAT so the user must be able to use the main screen features while a tutorial screen is open.
In the current implementation tutorial screens block the main screens event loop.