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

Graph file #60

Closed
wants to merge 6 commits into
base: master
from

Conversation

1 participant
@tiagofilipe12
Member

tiagofilipe12 commented Jul 3, 2017

This pull writes a graphson.json to file in the directory where the pipeline is run.

tiagofilipe12 added some commits Jun 27, 2017

Transfered experimental branch for development of graph (#59)
* added a manifest file output within lifecycle

* added a way to create a manifest file with the log of folders and tasks generated

* added more information on the manifest log

* changed the way folder was being reported to full paths

* changed variable from string to object and modified it to NDJSON in output file

* replaced if statement with lodash.get.

* Added a simple hack to output graphson like graph object
arranged a way to output a graphson object that can be properly used …
…to generate a visualization of the graph

@tiagofilipe12 tiagofilipe12 self-assigned this Jul 3, 2017

@tiagofilipe12 tiagofilipe12 requested a review from thejmazz Jul 3, 2017

@tiagofilipe12

This comment has been minimized.

Member

tiagofilipe12 commented Jul 3, 2017

Maybe we should wait for @bmpvieira opinion on this? Because of the many deletes with params from tasks generating a vertex.

@tiagofilipe12

This comment has been minimized.

Member

tiagofilipe12 commented Jul 5, 2017

After our talk with @bmpvieira , maybe we can merge this? However travis is failing because of:


     Error: timeout of 2000ms exceeded. Ensure the done() callback is being called in this test.```

@tiagofilipe12 tiagofilipe12 referenced this pull request Jul 5, 2017

Closed

Summary file #57

@tiagofilipe12

This comment has been minimized.

Member

tiagofilipe12 commented Jul 7, 2017

This was replaced by branch viz in PR #61

@tiagofilipe12 tiagofilipe12 deleted the graph branch Jul 7, 2017

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