-
Notifications
You must be signed in to change notification settings - Fork 52
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor: make the grapher usable as a library #45
Comments
Merged
The code is now cleaner. Will update the Readme with an example to integrate this in an existing project. Meanwhile, you can take how the CLI works as an example. |
For those following this issue: while the code can be used in another project, keep in mind that the internal API is not stable yet. I will likely refactor the internal of the grapher in the following versions. The Parser should not change that much. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I wrote most of the grapher almost three years ago in 2017 and lot of things happened since 馃槃
Currently, the Grapher is tightly coupled to the CLI. This makes it difficult to use it as a library inside another project. Also the
Grapher
is a little bit complicated: thinking of splitting it. I also want to make the graph customizable by not hardcoding the shapes, layout and make the post processor more extensible.So this issue is here to track the related changes. The outcome of these changes will be the version v1.0.0 of the grapher?
The text was updated successfully, but these errors were encountered: