Skip to content
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

docs: document the way to load samples #23

Merged
merged 6 commits into from
Jul 25, 2022
Merged

docs: document the way to load samples #23

merged 6 commits into from
Jul 25, 2022

Conversation

alexgarel
Copy link
Member

What

Part of

@alexgarel alexgarel requested a review from a team as a code owner July 22, 2022 14:25
alexgarel referenced this pull request Jul 22, 2022
* there is now a `is_before` relationship instead of block_index
* change the way to store tags and tags ids and uniformized among node
  entries
* add labels to nodes
* add a footer node
@aadarsh-ram aadarsh-ram changed the title doc: document the way to load samples docs: document the way to load samples Jul 22, 2022
in taxonomy parsing spec and in sample json for neo4j.
Copy link
Collaborator

@aadarsh-ram aadarsh-ram left a comment

Choose a reason for hiding this comment

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

Hi @alexgarel, these are just a few small changes I noticed.

doc/how-to-guides/develop-with-neo4j.md Outdated Show resolved Hide resolved
doc/how-to-guides/develop-with-neo4j.md Show resolved Hide resolved
doc/how-to-guides/develop-with-neo4j.md Outdated Show resolved Hide resolved
doc/how-to-guides/develop-with-neo4j.md Outdated Show resolved Hide resolved
doc/how-to-guides/develop-with-neo4j.md Outdated Show resolved Hide resolved
doc/how-to-guides/develop-with-neo4j.md Outdated Show resolved Hide resolved
@BryanH01
Copy link
Collaborator

Ah i forgot but, should the src_position you mentioned also be in the json file or it's only for the database ?

@alexgarel
Copy link
Member Author

Ah i forgot but, should the src_position you mentioned also be in the json file or it's only for the database ?

I'm not sure what you are talking about @BryanH01

@aadarsh-ram
Copy link
Collaborator

Ah i forgot but, should the src_position you mentioned also be in the json file or it's only for the database ?

@BryanH01 I think the Parser needs to generate a JSON according to the new format @alexgarel has created, so that it can be directed loaded into Neo4J. So, the property "src_position" would be used for debugging within the Parser program, but not stored in the JSON. @alexgarel @BryanH01 is my intiution correct?

@BryanH01
Copy link
Collaborator

Ah I'm sorry, you talk about it here when reviewing

@aadarsh-ram aadarsh-ram linked an issue Jul 22, 2022 that may be closed by this pull request
@BryanH01
Copy link
Collaborator

@BryanH01 I think the Parser needs to generate a JSON according to the new format @alexgarel has created, so that it can be directed loaded into Neo4J. So, the property "src_position" would be used for debugging within the Parser program, but not stored in the JSON. @alexgarel @BryanH01 is my intiution correct?

I'm sorry, I'm not being clear, i was talking about the test-neo4j.json file. The parser that I'm writing directly add the data in neo4j, I don't generate json.

@alexgarel
Copy link
Member Author

yes @aadarsh-ram , @BryanH01 is generating the Neo4J data directly, and that's what we want.

The json file is just for you to have some data to start with, and also was really useful at defining data structure more precisely.

@aadarsh-ram
Copy link
Collaborator

@alexgarel @BryanH01 Really sorry! Got a little bit confused with the programs. Thanks for the clarification!

@alexgarel
Copy link
Member Author

@BryanH01, you're right, I should add the "src_position" stuff to the spec, also in the sample to be constistent :-)

I forgot about it !

Will do.

@alexgarel
Copy link
Member Author

@BryanH01 I added the src_position.

* preceding *comments lines* in a `preceeding_lines` property
* preceding *comments lines* in a `preceding_lines` property
* the start of the entry block in a property `src_position`. First line is 1.
Comments (aka `preceding_lines`) are not accounted as block start, but for footer and header.

We also add a `is_before` link between nodes, to keep order in which nodes where found in the file.
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
We also add a `is_before` link between nodes, to keep order in which nodes where found in the file.
We also add a `is_before` link between nodes, to keep order in which nodes were found in the file.

@alexgarel alexgarel merged commit baeb2dd into main Jul 25, 2022
@alexgarel alexgarel deleted the doc-loadjson branch July 25, 2022 13:29
@alexgarel alexgarel restored the doc-loadjson branch July 25, 2022 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Add sample neo4j data
3 participants