Skip to content

Update README.md - #24

Open
Ennego wants to merge 1 commit into
NationalGalleryOfArt:mainfrom
Ennego:patch-2
Open

Update README.md#24
Ennego wants to merge 1 commit into
NationalGalleryOfArt:mainfrom
Ennego:patch-2

Conversation

@Ennego

@Ennego Ennego commented Oct 27, 2025

Copy link
Copy Markdown
Contributor

Add note about 4096 pixels on a side for IIIF images referenced in dataset.

Add note about 4096 pixels on a side for IIIF images referenced in dataset.
@nga-automation

Copy link
Copy Markdown
Collaborator

Hi @Ennego — thanks for this. Heads up that this PR now conflicts with main and is based on an older version of the README. As part of #30 we removed the audio/video media tables (media_items, media_relationships) from the open data program, and the README paragraph this PR edits was rewritten to reference images only (it no longer mentions "other media (such as audio and video files)"). Merging as-is would re-introduce that retired wording.

Could you please rebase onto the current main, re-apply your change on top of the updated text, and resubmit? Happy to review once it's refreshed. Thanks!

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.

2 participants