added `mmif`, `mmif source`, `mmif rewind` commands migrated from `clams-python` SDK #317

keighrim · 2025-06-04T23:47:24Z

addresses #316 .

* cleaned requirements.txt * renames serializer package * added vocab package

… not implemented

…sion of mmif-python)

* clams library does not explicitly imports mmif library * clams-python version is now specified in a separate file (not in setup.py) * mmif-python version is unspecified to enable using latest dev versions (with pip install --pre) * added missing files to sdist

…ded change_metadata method

…test convention

…efault parameters are used

…t` classes from example app

- changed rewinder file name taking off mmif_. - changed _delete_last function name to be single underscored. - added CLI argument for number of views to rewind. - changed read_mmif from json.load() -> file.read()

* optimized MMIF file I/O * added rewinding by number of apps (opposed to number of views, not exposed in CLI)

…hs are suppored

…opts

…ng guildlines

… scheme is passed as a part of location string

…to 316-migrate-cli-fromclams-tommif

codecov · 2025-06-04T23:50:42Z

Codecov Report

Attention: Patch coverage is 62.96296% with 80 lines in your changes missing coverage. Please review.

Project coverage is 88.64%. Comparing base (8a6a942) to head (11015e8).
Report is 58 commits behind head on develop.

Files with missing lines	Patch %	Lines
mmif/utils/cli/rewind.py	44.82%	32 Missing ⚠️
mmif/utils/cli/source.py	77.04%	28 Missing ⚠️
mmif/__init__.py	41.17%	20 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #317      +/-   ##
===========================================
- Coverage    92.59%   88.64%   -3.95%     
===========================================
  Files           10       13       +3     
  Lines         1405     1621     +216     
===========================================
+ Hits          1301     1437     +136     
- Misses         104      184      +80

Flag	Coverage Δ
unittests	`88.64% <62.96%> (-3.95%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

keighrim and others added 30 commits October 3, 2018 15:13

"clams" is now a pakcage, not a module

aed88ae

added restifier, updated other modules

021122a

Many updates for SDK

94202cf

* cleaned requirements.txt * renames serializer package * added vocab package

MmifObject (and attribs) now falls back to __str__() when serialize()…

3fbeba3

… not implemented

separating mmif-python package

c46f2d1

rename mmif.vocab import to mmif.vocabulary (based on current dev ver…

d775f6a

…sion of mmif-python)

added clams.source module with PipelineSource class

735ec37

Updated to use MMIF SDK 0.2.0

638271f

Added common_documents and common_metadata parameters to __init__; ad…

0e186c0

…ded change_metadata method

Added callable API and generator methods

2f21f0b

Merge branch 'master' into update-clams

a5fadcc

fixing minor issue in type checking, renamed test dir in line with py…

e28ff60

…test convention

clams.source.PipelineSource.__init__: swap in [] and {} for None if d…

fca2b61

…efault parameters are used

Added required metadata for spec 0.2.1

7ef2029

Added 'clams' CLI utility and implemented 'clams source' action

87b2266

fixed bugs in handling arguments for source cli

a49c3c6

adding '__version__' setter in setup.py and various getters in CLI

432b012

fixing typo

1f7340b

refactored CLI with argparser's subcommand parser

5b9de61

added prefix implementation

56384e7

now also tests argparsers

fdbc9c5

renamed clams.serve module to clams.app

a90e56e

Merge branch '31-prefix-sourcecli' into develop

eb5342b

added missing files

ae354c9

fixing unittest for version string

b79d7ef

flattening structure of runtime parameters, hiding Input and `Outpu…

cc7b9d6

…t` classes from example app

AppMetadata can be imported directly from clams package

42ec4ec

refactoring and fixing bugs in CLI

c60383c

removed usage of freezing methods in mmif objects

b68e1e8

leorabaumgarten and others added 19 commits June 27, 2023 12:10

modified and added tests after scheme support

3c183cf

added support for mixed schemes with --scheme

584f29c

updated test cases for source CLI

86b8c86

minor fix in testCLI for portability

f79320f

Initial implementation of mmif rewinder

e6425d6

Options are added: -o for output file name and -p for pretty printing.

23450f1

Review comments reflected:

017b170

- changed rewinder file name taking off mmif_. - changed _delete_last function name to be single underscored. - added CLI argument for number of views to rewind. - changed read_mmif from json.load() -> file.read()

updated rewinder module ...

480b6fa

* optimized MMIF file I/O * added rewinding by number of apps (opposed to number of views, not exposed in CLI)

merged source and rewind CLI under mmif_utils package

3e5dd74

updated rewinder CLI for stdin/out and picker mode argument

bcd5982

added a placeholder for test suite for rewinder

daa8340

basic rewinder tests

c93475b

fixed bugs in rewinder

7005d8e

minor cleanup of rewinder

57bc9da

clams source command now works on Windows, but still only POSIX pat…

3e29d1c

…hs are suppored

more helpful content for cli.py --help, cleaned up mmif rewinder cli …

fd1ada1

…opts

clams source --help now displays develop-provided location formatti…

f44d668

…ng guildlines

fixed source cmd returns location with duplicate URI schemes when the…

2295bfe

… scheme is passed as a part of location string

Merge remote-tracking branch 'origin/mmif-utils-from-clams-python' in…

eed2a81

…to 316-migrate-cli-fromclams-tommif

clams-bot added this to infra Jun 4, 2025

github-project-automation bot moved this to Todo in infra Jun 4, 2025

keighrim force-pushed the 316-migrate-cli-fromclams-tommif branch from 2e35843 to 2dff439 Compare June 5, 2025 11:54

finalized migration of CLI modules from clams-python

11015e8

keighrim force-pushed the 316-migrate-cli-fromclams-tommif branch from 2dff439 to 11015e8 Compare June 5, 2025 19:11

keighrim merged commit 969f393 into develop Jun 5, 2025
1 of 3 checks passed

github-project-automation bot moved this from Todo to Done in infra Jun 5, 2025

keighrim deleted the 316-migrate-cli-fromclams-tommif branch June 5, 2025 19:27

keighrim mentioned this pull request Jun 5, 2025

migrate mmif_utils from clams-python package #316

Closed

keighrim mentioned this pull request Jul 23, 2025

releasing 1.1.0 #322

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

added `mmif`, `mmif source`, `mmif rewind` commands migrated from `clams-python` SDK #317

added `mmif`, `mmif source`, `mmif rewind` commands migrated from `clams-python` SDK #317

Uh oh!

keighrim commented Jun 4, 2025

Uh oh!

codecov bot commented Jun 4, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

added mmif, mmif source, mmif rewind commands migrated from clams-python SDK #317

added mmif, mmif source, mmif rewind commands migrated from clams-python SDK #317

Uh oh!

Conversation

keighrim commented Jun 4, 2025

Uh oh!

codecov bot commented Jun 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

added `mmif`, `mmif source`, `mmif rewind` commands migrated from `clams-python` SDK #317

added `mmif`, `mmif source`, `mmif rewind` commands migrated from `clams-python` SDK #317

codecov bot commented Jun 4, 2025 •

edited

Loading