Gemma icon indicating copy to clipboard operation
Gemma copied to clipboard

CLI cleanups

Open arteymix opened this issue 2 years ago • 2 comments

  • introduce an abstract class for the batch-processing related logic
  • avoid overriding buildOptions and processOptions in AbstractCLI subclasses more than once

A few more things to look into:

  • [ ] tabular output for batch processing to report success/failures
  • [ ] use @Autowired to inject beans in CLI tools & remove getBean()
  • [ ] use typed options wherever possible

arteymix avatar Jan 25 '23 19:01 arteymix

@ppavlidis you mentioned the need for a tabular output, this can fit in here with the new AbstractBatchProcessingCLI class.

arteymix avatar Apr 16 '23 17:04 arteymix

@ppavlidis you mentioned the need for a tabular output, this can fit in here with the new AbstractBatchProcessingCLI class.

Ok I will probably look at that this week

ppavlidis avatar Apr 16 '23 18:04 ppavlidis