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
must be filled in a coherent manner up to now. Otherwise the return values are "/". This is especially true, if a directory does not exist yet. Also must the experiment_name be the same as the second to last string in the experiment_dir.
Furthermore, there could be a bug with the autocomplete, making the first autocomplete "fake", since we already give it to start with. Upon subsequent calls, or when loading datasets, the behaviour is as expected though.
In alrecon/alrecon/components/alrecon.py
we have the function define_recon_dir_BEATS
def define_recon_dir_BEATS(self, number_heading_directories=5):
### first 4 elements are defining the root at BEATS, + '/' what gets stripped to ''
### but since we know the path for BEATS, we could also take a hardcoded thing
path_ = os.path.abspath(self.experiment_dir.value)
elements_path = path_.split(os.sep)[:number_heading_directories]
elements_path[0] = '/'
dir_recon_root = os.path.join(*elements_path)
dir_recon_root = f'{dir_recon_root}_recon'
return dir_recon_root
which is tuned to our BEATS workflows. It was created anticipating other configurations and the way how it is invoked should be revised.
The text was updated successfully, but these errors were encountered:
note that in recon_dir and cor_dir foo can be replaced by whatsoever, autocomplete does not care, takes the experiment_name + the name of the dataset and a tweaked recon.
could be a step towards a more robust and understandable procedure.
One problem probably lies in how data are recorded, another one in the config file itself. In yaml references to other fields might be possible though, to achieve e.g. something like
In the config file the lines:
must be filled in a coherent manner up to now. Otherwise the return values are "/". This is especially true, if a directory does not exist yet. Also must the experiment_name be the same as the second to last string in the experiment_dir.
Furthermore, there could be a bug with the autocomplete, making the first autocomplete "fake", since we already give it to start with. Upon subsequent calls, or when loading datasets, the behaviour is as expected though.
In
alrecon/alrecon/components/alrecon.py
we have the function
define_recon_dir_BEATS
which is tuned to our BEATS workflows. It was created anticipating other configurations and the way how it is invoked should be revised.
The text was updated successfully, but these errors were encountered: