con / con/mechababs

BIDS-validate derivatives in finalize (deno validator, provenance-tracked)

Open
#36 0 comments 0 reactions 0 assignees View on GitHub
automation
Dominant language
Python
Stars
1
Forks
4
Avg merge
15h 39m
Merged PRs (30d)
24

Description

# BIDS-validate derivatives in finalize (deno validator, provenance-tracked)

## TODO check on bids-apps existing validation?

bids-apps probably already run the bids validator internally — but it's unclear
whether **before** the run, **after**, or **both**. Verify before adding a
redundant validation pass (and decide raw-input vs derivative-output scope).

## Problem

mechababs-produced derivatives should be BIDS-validated, with the report
captured as a provenance-tracked output, using the actively-maintained
**deno** bids-validator (not the legacy Node version).

## The tool — `bids-validator-deno`

- Add `bids-validator-deno` to `requirements.txt` / environment spec.
- Ensure it's available inside the container image.

## The wiring — `datalad run` it in finalize

- Finalize step `datalad run`s the validator so the validation **report is a
provenance-tracked output**.
- **Keep the validation output even when validation FAILS** — persist
stdout/stderr to a well-known path regardless of exit code, so the failure
itself is inspectable after the fact.
- Add **Pygments** for colorized local viewing.

## Cross-ref

Relates to the **bids-validator hook** in the hooks design (`PennLINC/babs#365`)
— PR1's `pre_app` example replacing `--skip-bids-validation`, and the `post_run`
validator-on-derivatives form. This issue is the mechababs-side finalize
validation; the hook is the babs-side mechanism — reconcile if hooks land.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.