You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The relative paths in BART.cfg in the example and demo directory are misleading. I.E. func = hack BARTfunc ../../code/ specify a path relative to the directory in which the function which reads the config file resides rather than the directory which contains the config file (i think, the path may be broken all together). By this i mean within the BART/examples/ directory the BART.cfg file specifies the path ../../code/ for the location of the BARTfunc, however from this location in the directory structure the path to code would be ../code . Specifying the path in this manor may be necessary for running the program, but it is not obvious to someone reading the code to use or modify it. If possible use some variable to specify a base directory and do things relative to that variable, or put a code comment helping the reader understand where that function is contained.
The text was updated successfully, but these errors were encountered:
The relative paths in BART.cfg in the example and demo directory are misleading. I.E. func = hack BARTfunc ../../code/ specify a path relative to the directory in which the function which reads the config file resides rather than the directory which contains the config file (i think, the path may be broken all together). By this i mean within the BART/examples/ directory the BART.cfg file specifies the path ../../code/ for the location of the BARTfunc, however from this location in the directory structure the path to code would be ../code . Specifying the path in this manor may be necessary for running the program, but it is not obvious to someone reading the code to use or modify it. If possible use some variable to specify a base directory and do things relative to that variable, or put a code comment helping the reader understand where that function is contained.
The text was updated successfully, but these errors were encountered: