Skip to content

Add instructions for embedding images within the DLP Description field - #200

Closed
kabilar wants to merge 3 commits into
dandi:masterfrom
kabilar:images
Closed

Add instructions for embedding images within the DLP Description field#200
kabilar wants to merge 3 commits into
dandi:masterfrom
kabilar:images

Conversation

@kabilar

@kabilar kabilar commented Apr 17, 2025

Copy link
Copy Markdown
Member

@kabilar
kabilar requested a review from bendichter April 17, 2025 23:39
@bendichter

Copy link
Copy Markdown
Member

This is a cool idea! But do we want to allow and encourage users to link to external images? That conflicts a bit with the idea that we want to metadata to be persistent and unchanging for a given dandiset version. What if we somehow stored these images any linked image in a place we control?

@kabilar

kabilar commented Apr 18, 2025

Copy link
Copy Markdown
Member Author

This is a cool idea! But do we want to allow and encourage users to link to external images? That conflicts a bit with the idea that we want to metadata to be persistent and unchanging for a given dandiset version. What if we somehow stored these images any linked image in a place we control?

That's a fair point. Perhaps we could have users upload images to the Dandiset with the dandi-cli and then allow for relative links in the Description field. We can discuss further in dandi/dandi-archive#2299.

@kabilar kabilar closed this Apr 18, 2025
yarikoptic added a commit to yarikoptic/bids-specification that referenced this pull request Apr 18, 2025
Inspired by the success of the
bids-standard#1962 adding
formalization of `logs/` folder, and triggered by the use-case in DANDI
dandi/dandi-docs#200 seeking to add more elaborate
descriptions to the README (or more specifically README.md), I would like to
propose adding any extra documentation and associated artwork (images, videos,
etc) under `docs/` folder.  I do not think it is worth breaking down more (as
videos/) etc, since it could confuse with data types (videos of behavior or
alike).

Not sure if we would want to formalize inner structure anyhow. We could
potentially recommend using `docs/images/` subfolder for images.

Then README.md on top level could potentially use those images via
references to `docs/` paths.
@yarikoptic

Copy link
Copy Markdown
Member

yes, ideally we should allow for that at BIDS level as well, hence fresh

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.

Allow for embedding images within the DLP Description field

3 participants