RENCI / pds

PDS application (tx-router plugin)
MIT License
1 stars 0 forks source link

[Feature] cli.py is in mapper plugin, should be in own repo #242

Closed krobasky closed 3 years ago

krobasky commented 3 years ago

What problem(s) do you aim to solve? Need to make a sustainable, stand-alone cilent tool to support model developers in a research arena.

What feature solves it and why?

The tool should use the code that's currently in https://github.com/RENCI/pdspi-mapper-parallex-example/blob/master/cli.py, and should be in its own repository, similar to https://github.com/RENCI/pds-release in order to meet the following requirements:

Close the this ticket when:

This is the final product for #216

krobasky commented 3 years ago

Let's remove this requirement:

krobasky commented 3 years ago

cli.py now has it's own repo: https://github.com/RENCI/pds-cli-cp/ But it still doesn't use the pds interface; it instead calls the fhir and mapper plugins directly.

This needs more user feedback before we can finish the redesign.

krobasky commented 3 years ago

It's OK to have cli.py to call the mapper and fhir appliances directly if we create an OMOP mapper because then the mapper/fhir appliances will be generic enough while also specific to the research data warehouse being used by the cli.py program. See #257