When discussing the Order Based Procflow (OBP), we decided that we should consolidate commandline functionality into the CLI. Therefore create_sector_image.py, and most other command line accessible functionality should sooner or later be transitioned into the GeoIPS CLI. This should be an easy change, and we'll likely add this merge this functionality as a geoips test sector <sector_name> call.
Background and Motivation
This issue came up in a discussion over the OBP #257.
Code to demonstrate issue
geoips/commandline/create_sector_image.py.
Checklist for Completion
[ ] Create a new CLI class under geoips_test.py called GeoipsTestSector
[ ] Move create_sector_image.py into the GeoipsTestSector
[ ] Delete create_sector_image.py and it's corresponding command create_sector_image in pyproject.toml
Requested Update
Description
When discussing the Order Based Procflow (OBP), we decided that we should consolidate commandline functionality into the CLI. Therefore
create_sector_image.py, and most other command line accessible functionality should sooner or later be transitioned into theGeoIPS CLI. This should be an easy change, and we'll likely add this merge this functionality as ageoips test sector <sector_name>call.Background and Motivation
This issue came up in a discussion over the OBP #257.
Code to demonstrate issue
geoips/commandline/create_sector_image.py.Checklist for Completion
geoips_test.pycalledGeoipsTestSectorcreate_sector_image.pyinto theGeoipsTestSectorcreate_sector_image.pyand it's corresponding commandcreate_sector_imageinpyproject.toml