ECP-CANDLE / candle_lib

MIT License
1 stars 9 forks source link

add candle_data_dir to directory_from_parameters #11

Closed brettin closed 1 year ago

brettin commented 1 year ago

add CANDLE_DATA_DIR to output path

jmohdyusof commented 1 year ago

So commonroot ('Output' by default) should be appended to CANDLE_DATA_DIR, just like cache_subdir (which is basically the input directory). So 176 should look like 178, and I think the check should be more robust:

       if os.getenv("CANDLE_DATA_DIR") is not None:
            # outdir = os.getenv("CANDLE_DATA_DIR")
            outdir = os.path.abspath(os.path.join(os.getenv("CANDLE_DATA_DIR"), commonroot))
        else:
            outdir = os.path.abspath(os.path.join(".", commonroot))
rajeeja commented 1 year ago

Run pre-commit run --all-files