Skip to content

Commit 51e0e52

Browse files
committed
Remove maldives ecosystem mapping training pipelines.
These aren't used since the README already says to just run `model fit` directly and I think the train pipeline is kind of pointless since it's just one line and really should just reference the config file (so even if it isn't documented in a README, it should be easy to reproduce). This resolves #14. This also removes the class from the pipeline definitions so it just needs to be a function, the class was not used.
1 parent 72100cb commit 51e0e52

File tree

6 files changed

+9
-37
lines changed

6 files changed

+9
-37
lines changed
Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
11
"""Forest loss driver classification project."""
22

33
from .predict_pipeline import (
4-
PredictPipelineConfig,
54
predict_pipeline,
65
select_best_images_pipeline,
76
)
87

98
workflows = {
10-
"predict": (PredictPipelineConfig, predict_pipeline),
11-
"select_best_images": (None, select_best_images_pipeline),
9+
"predict": predict_pipeline,
10+
"select_best_images": select_best_images_pipeline,
1211
}

rslp/landsat_vessels/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,5 @@
33
from .predict_pipeline import predict_pipeline
44

55
workflows = {
6-
"predict": (None, predict_pipeline),
6+
"predict": predict_pipeline,
77
}

rslp/main.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ def main():
2121
args = parser.parse_args(args=sys.argv[1:3])
2222

2323
module = importlib.import_module(f"rslp.{args.project}")
24-
workflow_fn = module.workflows[args.workflow][1]
24+
workflow_fn = module.workflows[args.workflow]
2525
jsonargparse.CLI(workflow_fn, args=sys.argv[3:])
2626

2727

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,10 @@
11
"""Maldives ecosystem mapping project."""
22

3-
from rslp.config import BaseTrainPipelineConfig
4-
5-
from .data_pipeline import DataPipelineConfig, data_pipeline
3+
from .data_pipeline import data_pipeline
64
from .predict_pipeline import maxar_predict_pipeline, sentinel2_predict_pipeline
7-
from .train_pipeline import maxar_train_pipeline, sentinel2_train_pipeline
85

96
workflows = {
10-
"data": (DataPipelineConfig, data_pipeline),
11-
"train_maxar": (BaseTrainPipelineConfig, maxar_train_pipeline),
12-
"train_sentinel2": (BaseTrainPipelineConfig, sentinel2_train_pipeline),
13-
"predict_maxar": (None, maxar_predict_pipeline),
14-
"predict_sentinel2": (None, sentinel2_predict_pipeline),
7+
"data": data_pipeline,
8+
"predict_maxar": maxar_predict_pipeline,
9+
"predict_sentinel2": sentinel2_predict_pipeline,
1510
}

rslp/maldives_ecosystem_mapping/train_pipeline.py

Lines changed: 0 additions & 22 deletions
This file was deleted.

rslp/sentinel2_vessels/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,5 @@
33
from .predict_pipeline import predict_pipeline
44

55
workflows = {
6-
"predict": (None, predict_pipeline),
6+
"predict": predict_pipeline,
77
}

0 commit comments

Comments
 (0)