NRLMMD-GEOIPS / geoips

Main Geolocated Information Processing System code base with basic functionality enabled.
https://nrlmmd-geoips.github.io/geoips/
Other
13 stars 10 forks source link

Discuss whether or not we should implement ``CLI Plugins`` #631

Closed evrose54 closed 1 week ago

evrose54 commented 2 weeks ago

Requested Update

Description

Per Jeremy's comment on #465.

What I'd like to do in the long-run, though, is add the ability to have CLI plugins.

We should discuss whether CLI plugins are needed. Maybe they'd be useless after creating the order-based procflow, though.

I'm not sure how we'd exactly implement CLI Plugins, but I could see their use cases. It might simplify the CLI a bit more and could be used internally throughout other locations of the code. We'd need to implement an internal version of the CLI that'd output dictionaries, lists, etc. rather than sys.stout based output, but that wouldn't be too hard of a refactor.

This is largely just a discussion issue to determine whether or not this factor would be worth it, and what benefits would stem from this change.

Background and Motivation

This comes from this comment on #465.

Alternative Solutions

Leave the code as-is.

Code to demonstrate issue

geoips.commandline.geoips_run:GeoipsRunDataFusion

Checklist for Completion

jsolbrig commented 1 week ago

I don't think that I see a use case for this anymore. Closing.