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

Add parsing function for dot files #18

Open
Porlockzzz opened this issue Oct 31, 2022 · 3 comments
Open

Add parsing function for dot files #18

Porlockzzz opened this issue Oct 31, 2022 · 3 comments

Comments

@Porlockzzz
Copy link

Nice work!

But I have a requirement to read the generated .dot, modify the nodes/edges of the .dot file, and then generate a new .dot

It seems like a new parsing function needs to be added.

Looking forward to your reply.

Best wishes,
Porlockzzz

@alexandresalome
Copy link
Owner

Hey, thank you ❤️

To achieve parsing of DOT files, there is some preparation work to be done:

  • Implement the comments (//, /* */ and #)
  • Introduce a new Document object, as the comments and spaces can be before or after the graph object.

Given that this doable work is done, a parser would still have some limitations:

  • The graph would be re-formatted (custom spacing lost)
  • New instructions would be added at the end of the graphs

I don't know what you require exactly for your specific case, but if that's OK for you with those limitations I can maybe work on that in the coming weeks.

@Porlockzzz
Copy link
Author

Hey, thank you heart

To achieve parsing of DOT files, there is some preparation work to be done:

  • Implement the comments (//, /* */ and #)
  • Introduce a new Document object, as the comments and spaces can be before or after the graph object.

Given that this doable work is done, a parser would still have some limitations:

  • The graph would be re-formatted (custom spacing lost)
  • New instructions would be added at the end of the graphs

I don't know what you require exactly for your specific case, but if that's OK for you with those limitations I can maybe work on that in the coming weeks.

These restrictions do not affect my needs, thanks for your work!🥰

@alexandresalome
Copy link
Owner

Comments have been implemented in the model, I need to introduce now a Document class and I'll start working on the parser, properly :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants