Refine dataset docs overview cards and summary pages#1025
Merged
bruAristimunha merged 9 commits intodevelopfrom Mar 6, 2026
Merged
Refine dataset docs overview cards and summary pages#1025bruAristimunha merged 9 commits intodevelopfrom
bruAristimunha merged 9 commits intodevelopfrom
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Remove the standalone Page Views section from above Class Labels and merge it as a single rich row inside the Citation & Impact card. The new row uses a mini two-column grid (metrics text + sparkline) that stays consistent with the existing label/value pattern.
- Remove PapersWithCode column from summary CSVs, base.py, docs - Add collapsible Overview teaser panel between buttons and quickstart in the dataset snapshot card, showing the first ~800 chars of the docstring with reST-to-HTML formatting (bold, lists, code) - "Show more" expands full description; "Open in Overview tab" scrolls to and activates the Overview tab below
…riable - Compile _RST_INLINE_RE, _RST_FOOTNOTE_RE, _RST_LIST_SPLIT_RE at module level instead of inside per-call functions - Add --ds-font-mono custom property and replace 9 occurrences of the inline monospace font stack
bruAristimunha
force-pushed
the
feedback-katie
branch
from
42ff89a to
bf8bac6
Compare
The global autodoc_default_options {"inherited-members": False} was
silently overriding the per-directive :inherited-members: option injected
by the source_read hook. Sphinx's option merging always replaces
directive-level values with boolean global defaults, so the hook had no
effect. Removing the redundant setting fixes it — Sphinx already hides
inherited members by default when the option is absent.
Also adds :show-inheritance: to display the "Bases: ..." line.
Switch table-layout from fixed to auto and remove the hard 36% first-column width on screens narrower than 768px so columns size to content instead of overlapping.
BDF is not a valid format for the EEG datatype in the BIDS specification. The BIDS validator rejects .bdf files with "Not allowed for EEG datatype".
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Validation