Saving profile modifies the source code

[mshavliuk]

Description

Whenever I make any changes and click “Save Profile” in the Mesh tool, it updates the source code of the app, which is tracked by git so the git pull will not work anymore unless I stash or discard changes. This looks like a flaw and prevents from saving profiles while updating the project.

Suggestion

I would suggest to store changed profiles similarly, as we handle saving projects - suggest user to store them as a separate file or embed them inside the project .mat file

Screenshots

Screen.Recording.2023-05-08.at.20.57.01.mov

[mshavliuk]

Similar is applicable for all other profiles that could be modified from Settings menu

[SeSodesa]

If we want to keep the loading of available lead field functions dynamic like this, I think the process could be improved by generating the available menu items based on what is found in a certain folder (or folders), say m/forward_simulation/lead_field. The names of the menu items could be the file names without an extension, and underscores _ changed to spaces . If the helper functions related to lead field construction are to be kept in that same folder, the loaded functions could be chosen based on whether there exists a .txt file of the same name as the function, with a description of the function in the folder. My point is, it is a bit backwards to have to write a separate config file in some completely unrelated folder, that is also checked into the repository, when the information can already be parsed in some other manner from static text.

But I would also question the need for dynamic loading in this case. Lead field construction is such a central part of Zeffiro, that I think these menu items should be static. If there were a million functions, I might understand the need for dynamic loading based on what is already in some folder, but this is not (yet) the case.

[mshavliuk]

But I would also question the need for dynamic loading in this case. Lead field construction is such a central part of Zeffiro, that I think these menu items should be static. If there were a million functions, I might understand the need for dynamic loading based on what is already in some folder, but this is not (yet) the case.

I agree 👍, cutting off the unnecessary features is a good approach, which would allow to avoid overengineering and focus mental resources on core functionality.