Skip to content

Ensure term consistency: nodes vs vertices #730

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Ensure term consistency: nodes vs vertices #730

wants to merge 6 commits into from

Conversation

nerpaula
Copy link
Contributor

@nerpaula nerpaula commented Jun 30, 2025 •
edited by Simran-B
Loading

Description

TODO:

  • Apply to 3.12
  • GraphML UI label to be changed? Select Vertex Collection(s)
  • NetworkX integrations: replace term in code example? (Creates an inconsistency between parameter names of which some must remain as they are whereas others can be freely changed)
  • Go code example uses MyVertexNode and MyEdgeLink, change both? (MyNode, MyEdge)

Upstream PRs

  • 3.10:
  • 3.11:
  • 3.12:
  • 3.13:

@cla-bot cla-bot bot added the cla-signed label Jun 30, 2025
@arangodb-docs-automation arangodb-docs-automation
Copy link
Contributor

Deploy Preview Available Via
https://deploy-preview-730--docs-hugo.netlify.app

Simran-B added 5 commits August 5, 2025 12:32
@Simran-B
Merge branch 'main' of https://github.com/arangodb/docs-hugo into nod…
00c1812 
…es-vertices
@Simran-B
Clean up feature list
48df435
@Simran-B
Remove paragraph about X and Y Vertex Collections
468277f 
It's apparently attribute suffixes, not collections, and the X and Y got lost already so this didn't really help
@Simran-B
Replace "vertex" with "node"
818bc02 
Exceptions: API endpoints/methods, vertex-centric indexes, measures like vertex-centrality
@Simran-B
Merge branch 'main' of https://github.com/arangodb/docs-hugo into nod…
1e7c75e 
…es-vertices
@Simran-B Simran-B marked this pull request as ready for review August 12, 2025 08:43
nerpaula
nerpaula commented Aug 13, 2025
View reviewed changes
Copy link
Contributor Author

@nerpaula nerpaula left a comment

Choose a reason for hiding this comment

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

LGTM! Let's change the GraphML UI label in a separate PR, there's one screenshot involved as well so we need to wait for frontend changes. For NetworkX and Go code examples, I'd say to leave them as they are.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nerpaula @Simran-B