Skip to content

refactor(asr-worker): save transcriptions to artifact filesystem#14

Open
ClemDoum wants to merge 2 commits intomainfrom
feature(asr-worker)/save-results
Open

refactor(asr-worker): save transcriptions to artifact filesystem#14
ClemDoum wants to merge 2 commits intomainfrom
feature(asr-worker)/save-results

Conversation

@ClemDoum
Copy link
Copy Markdown
Contributor

@ClemDoum ClemDoum commented Mar 26, 2026

Description

Addresses #10 and #9.

Note

Will require some caul updates to pass e2e test.

Changes

asr-worker

Changed

  • defined transcription outputs independently of caul objects to ensure stable worker API
  • refactored ASRActivities to read input data from the worker audio_root, save intermedite results in the worker workdir and save final results in the worker artifacts_dir
  • update ASRResponse to return the number of transcribed files

datashare-python

Added

  • add a number of utils to read/write artifact metadata as well as creating artifact directories and filesystem tree structure resilient to a large number of files

.github

Added

  • activated unit and linting test for asr-worker

@ClemDoum ClemDoum force-pushed the feature(asr-worker)/save-results branch from 36dff47 to 679f718 Compare March 26, 2026 14:29
@ClemDoum ClemDoum force-pushed the feature(asr-worker)/save-results branch from 679f718 to dec1868 Compare March 27, 2026 08:29
@ClemDoum ClemDoum requested a review from winsomeglint March 27, 2026 09:39
@ClemDoum ClemDoum force-pushed the feature(asr-worker)/save-results branch 3 times, most recently from 33bce0e to 7e22220 Compare March 27, 2026 10:50
@ClemDoum ClemDoum marked this pull request as ready for review March 27, 2026 10:58
@ClemDoum ClemDoum force-pushed the feature(asr-worker)/save-results branch from 7e22220 to 6a3e192 Compare March 27, 2026 11:06
@ClemDoum ClemDoum force-pushed the feature(asr-worker)/save-results branch from 6a3e192 to 2d206b2 Compare March 27, 2026 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant