bug: update keep matlab flag to work as documentation indicates

[ahurli]

Purpose

Fix the default behavior of the keep-matlab flag to match the documentation.

What is the code doing?

Currently if the --keep-matlab flag is used, the matlab files will be deleted and the default is to keep them. Switching this line will switch the default behavior so that these files are only kept if the flag is used.

Where to Look

The code changed is in extract_data.py.

Time to Review

5 min.

[danielolsen]

Previously, after parsing the args with the flag not set:
Namespace(end_date=None, execute_dir=None, frequency='H', keep_matlab=True, matlab_dir=None, output_dir=None, scenario_id='1601', start_date=None)
With the latest code:
Namespace(end_date=None, execute_dir=None, frequency='H', keep_matlab=False, matlab_dir=None, output_dir=None, scenario_id='1601', start_date=None)
And the behavior works as expected.

[danielolsen]

Thanks for the quick fix!