Skip to content

Updated images #1210

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.

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

Open
wants to merge 31 commits into
base: cypher-25
Choose a base branch
from
Open

Updated images #1210

wants to merge 31 commits into from

Conversation

lidiazuin
Copy link
Contributor

No description provided.

Copy link
Collaborator

@JPryce-Aklundh JPryce-Aklundh left a comment

Choose a reason for hiding this comment

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

Thanks @lidiazuin.
The images look good, except for:

  • graph_clause_where.svg (the replacement replaces the old image)
  • patterns-equijoins-solution_2.svg (I think the different solutions are less clear in this new format).

Also, how are you batching these updates? I see, e.g., that this PR replaces some but not all of the svgs in the Patterns chapter? Are those not replaced here not ready yet?

@@ -678,7 +678,7 @@ RETURN type(r) AS type, m.title AS movie

The result shows that he has 13 outgoing relationships connected to 12 different `Movie` nodes (12 have the `ACTED_IN` type and one has the `DIRECTED` type).

image::introduction_example1.svg[width="500",role="middle"]
image::introduction-example1.svg[A Tom Hanks nodes connecting to Movie nodes via acted in relationships,width=600,role=popup]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
image::introduction-example1.svg[A Tom Hanks nodes connecting to Movie nodes via acted in relationships,width=600,role=popup]
image::introduction-example1.svg[A Tom Hank Person node connected to Movie nodes via acted in relationships,width=600,role=popup]

Co-authored-by: Jens Pryce-Åklundh <[email protected]>
@lidiazuin
Copy link
Contributor Author

lidiazuin commented Mar 12, 2025

  • graph_clause_where.svg (the replacement replaces the old image)

I don't understand your point here?

  • patterns-equijoins-solution_2.svg (I think the different solutions are less clear in this new format).

Isn't it the same image, only the nodes are organized differently? Maybe you can talk to Petar and explain to him what improvements could be made.

  • Also, how are you batching these updates? I see, e.g., that this PR replaces some but not all of the svgs in the Patterns chapter? Are those not replaced here not ready yet?

I'm adding all the images that Petar finished and sent to me in the first batch. I don't know in what order he's working with the images, but I would guess it's not after chapters since we have some missing, like the ones you pointed out.

@JPryce-Aklundh
Copy link
Collaborator

JPryce-Aklundh commented Mar 12, 2025

  1. The WHERE graph was updated in cypher-25. Your image is a replacement of the image in dev/5.x.
    I pointed out the difference in the google doc I shared with you and Petar.

  2. They are not the same. The new one only differentiates the nodes within the cycle by the width of the rel - the old one used a different color for the rels outside the cycle, which, I think, is clearer.
    https://neo4j.com/docs/cypher-manual/current/patterns/non-linear-patterns/
    I will ask Petar

  3. Ok, I was under the impression that all images had been completed (apart from the new ones in cypher 25, such as the WHERE graph)

@lidiazuin
Copy link
Contributor Author

Would you like to revert the change in these two images and merge to publish the new ones or wait?

@JPryce-Aklundh
Copy link
Collaborator

where_clause.svg should be reverted, but I actually think it might be better to wait adding the images in the Patterns section until they are all done (i.e. all "patterns... .svg" files).

@recrwplay
Copy link
Contributor

This PR includes documentation updates
View the updated docs at https://neo4j-docs-cypher-1210.surge.sh

Updated pages:

…of images that are not being used but were updated
@neo-technology-commit-status-publisher
Copy link
Collaborator

This PR includes documentation updates
View the updated docs at https://neo4j-docs-cypher-1210.surge.sh

Updated pages:

@@ -2,6 +2,12 @@
[[syntax-and-semantics]]
= Syntax and semantics

//Check Mark
Copy link
Collaborator

Choose a reason for hiding this comment

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

What does this do? (Preview isnt working)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It replaces the current emoji ✅ for the checkmark we use in docs, so it should look like this instead:

Screenshot 2025-04-04 at 11 25 09

lidiazuin and others added 15 commits May 21, 2025 09:44
…of images that are not being used but were updated
Publish from all the branches
Co-authored-by: AlexicaWright <[email protected]>
Co-authored-by: Jens Pryce-Åklundh <[email protected]>
Co-authored-by: David Oliver <[email protected]>
Co-authored-by: Arne Fischereit <[email protected]>
Co-authored-by: Tobias Johansson <[email protected]>
Co-authored-by: Gemma Lamont <[email protected]>
Co-authored-by: lidiazuin <[email protected]>
Co-authored-by: Therese Magnusson <[email protected]>
Co-authored-by: JPryce-Aklundh <[email protected]>
Co-authored-by: Louise Söderström <[email protected]>
Co-authored-by: Jack Waudby <[email protected]>
Co-authored-by: Satia Herfert <[email protected]>
Co-authored-by: Gustav Hedengran <[email protected]>
Feature merged in 5.3 but hidden behind feature flag, the flag was turned true by default in 5.4.
Original PR: neo4j#348

Fix unterminated example block in alias page.
This makes the example always return the same result by enforcing an
ordering, so that we don't need to skip testing its result (this PR
updates neo4j#405). The sense of the example is unaffected.
Original PR: neo4j#464

Cherry-pick excludes 5.6 specific content in original PR

Co-authored-by: Therese Magnusson <[email protected]>
JPryce-Aklundh and others added 10 commits May 21, 2025 11:35
Co-authored-by: Satia Herfert <[email protected]>
Co-authored-by: Hannes Sandberg <[email protected]>
Co-authored-by: Nacho Cordón <[email protected]>
Co-authored-by: Therese Magnusson <[email protected]>
Co-authored-by: Pontus Melke <[email protected]>
Co-authored-by: Jens Pryce-Åklundh <[email protected]>
Co-authored-by: Bastien Louërat <[email protected]>
Co-authored-by: Therese Magnusson <[email protected]>
Co-authored-by: Lasse Heemann <[email protected]>
This PR updates 5.x with updates from PRs into dev that were not
cherry-picked into 5.x at any point.

While it looks like a lot of commits were missing, in the end it amounts
to changes in 14 files, some of which are trivial and do not affect the
content or output.

I'd be happy to use 'squash and merge' to reduce this update to a single
commit in 5.x
…order on the subsections (neo4j#569)

Original PR: neo4j#568

authored-by: Therese Magnusson <[email protected]>
@neo4j-docops-agent
Copy link
Collaborator

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.

9 participants