Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ENH] Formalize presence of optional logs/ folder #1962

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Oct 17, 2024

  1. [ENH] Formalize presence of logs/ folder

    It is quite often desired to store logs, e.g. simply stdout/stderr from
    invocation of BIDS converters or other tools which were used to produce/change
    content in current BIDS datasset.  They quite often provide ultimate provenance
    information to troubleshoot odd or incorrect results.  But researchers, in my
    personal opinion, undervalue logs!  But hinting them on "standard" location for
    them, I think we could inspire more of relevant to dataset provenance metadata
    being collected and shared.  Dangers: might leak sensitive metadata, so may be
    wording should be adjusted to mention that they should be
    inspected/sensored.
    
    Shameless plug: consider using con-duct (https://github.com/con/duct) for your
    "logging needs".
    
    Note that I think that ideally the `logs/` should not include "derivative"
    data, such as e.g. output of `bids-validator`.  Those outputs should go under
    `derivatives/bids-validator-{version}/output.json` or alike.  (It might be the
    only reasonable "derivative" to recommend bundling with any "raw" or
    derived BIDS dataset).
    yarikoptic committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    1a50063 View commit details
    Browse the repository at this point in the history