Skip to content

Conversation

@AnandMukherjee2004
Copy link
Contributor

@AnandMukherjee2004
Copy link
Contributor Author

All checks have passed and there are no merge conflicts. This PR is ready for review and merge. Please let me know if there are any further suggestions or changes needed. Thank you!

@AnandMukherjee2004
Copy link
Contributor Author

hey, is there anyone!!
All checks have passed and there are no merge conflicts. This PR is ready for review and merge. Please let me know if there are any further suggestions or changes needed. Thank you!

@AnandMukherjee2004
Copy link
Contributor Author

hey @mroeschke @afeld ,
please look at this pr! Its pending from a long time.

@github-actions
Copy link
Contributor

This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this.

@github-actions github-actions bot added the Stale label Oct 20, 2025
Copy link
Contributor

@afeld afeld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, apologies for the slow response here. A few things:

  • Looks like some changes for #62314 were included — please remove, to keep this pull request focused.
  • Let's try and match the style of the existing diagrams, for consistency. Specifically:
    • Font — not sure what it is, off hand
    • The different grey fills
  • 01_table_dataframe.svg is used multiple places. Let's replace the existing file, for simplicity.
  • Please clean up the commit history, rebasing so there's just a single commit with the change.

Thanks!

@afeld
Copy link
Contributor

afeld commented Oct 20, 2025

Hey, actually, I missed that someone else had grabbed it, and has a pull request that's closer to ready: #62466 I appreciate your work on this!

@afeld afeld closed this Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DOC: Expand data table respresentation

2 participants