search

loinc / comp-loinc

Computational LOINC in OWL.
MIT License
13 stars 4 forks source link

(i) CLI updates, (ii) `makefile`, (iii) misc updates #23

Closed joeflack4 closed 2 years ago

joeflack4 commented 2 years ago

Updates

    - Added: makefile
    - Code style: Misc minor code style updates for PEP8 and other PEP compliance.
      - Comments at top of file
      - Spacing
      - Import reordering: (i) standard libary, (ii) external packages, (iii) imports from within package.
    - Documentation
      - Added some code comments.
      - Added CLI help text.
      - Updated formatting of README
    - Updates: CLI
      - Added ability to call the package directory: `python -m comp_loinc`
      - Added / updated help text
      - Added defaults
      - Updated types: str -> path for several
      - Added: Path types: requirement for some paths: (i) exists=True, (ii) writable=True
      - Added: Path types relative to absolute path resolution: resolve_path=True
      - Update: Changed arguments from positional to named.
      - Bugfix: Typo: reasponed -> reasoned
      - Update: 'reason' command: removed owl_directory, as the directory where merged and umerged OWL files was different but ambiguous.
      - Update: renamed directory: `owl_files` -> `owl_component_files`
joeflack4 commented 2 years ago

@putmantime I think this is ready for review / merging.