/bionode-watermill

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

Implemented a graph visualization #62

Merged
merged 30 commits into from Jul 8, 2017

Conversation

Reviewers

@thejmazz thejmazz

Assignees

@tiagofilipe12 tiagofilipe12

Labels
Projects
  
Done in Bionode Project Board
Milestone
No milestone
3 participants
@tiagofilipe12 @thejmazz @bmpvieira
@tiagofilipe12
Member

tiagofilipe12 commented Jul 7, 2017

This PR will add to master a graphical visualization accessible in localhost:8084 and an output file graphson.json that is a NDJSON file that can be interpreted as a summary file for the log of each pipeline runned. It is still missing the operationString, that we should fix in the future.

tiagofilipe12 added some commits Jun 27, 2017

@tiagofilipe12
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
Loading status checks…
285cc9f
@tiagofilipe12
Implemented test logs to start outputing to graphson from graph.js ge… 
…nerated object
Loading status checks…
70020f3
@tiagofilipe12
added vertices and edges to graphsonObj
Loading status checks…
524d2d5
@tiagofilipe12
arranged a way to output a graphson object that can be properly used … 
…to generate a visualization of the graph
Loading status checks…
1a78ea2
@tiagofilipe12
removed some sketchy code that it is not needed now.
Loading status checks…
8acd5e4
@tiagofilipe12
added output file for graphsonObj
Loading status checks…
9823556
@tiagofilipe12
started testing real-time graph visualization
a0aff17
@tiagofilipe12
Added a first implementation of real-time graph viz
ed6946b
@tiagofilipe12
reduced number of graph calls necessary
7c82bfd
@tiagofilipe12
added a tester version to update DOM for real-time graph
f6e7e54
@tiagofilipe12
added more elements to label in graph visualization
a530f4d
@tiagofilipe12
stringified input value that can be an object
4d9821a
@tiagofilipe12
cleaned code and added information on the type of vertex
aa9dd5f
@tiagofilipe12
commented the filter for type
bd83455
@tiagofilipe12
removed unused app.js
c10ead4
@tiagofilipe12
added direction of the workflow.
556e07f
@tiagofilipe12
passed server.listen to outside jsonifyGraph function
e1b90aa
@tiagofilipe12
Added labels to graph visualizationa and increased speed of the rende… 
…ring
0861f67
@tiagofilipe12
removed console.log and spaces in code
1270ba7
@tiagofilipe12
removed aplicator from collections.js
6889fce
@tiagofilipe12
commented code that was breaking the pipeline run
9bcd157
@tiagofilipe12
Merge remote-tracking branch 'origin/logger_updated' into viz
c51a447
@tiagofilipe12
removed manifest file log and replaced writeFile with writeFileSync
b35965b
@tiagofilipe12
Handled setMaxListeners for socket.io
d357bef
@tiagofilipe12
fixed bwa related tasks that was rendering an unhandled exception
17983dd
@tiagofilipe12
for now input pattern was added
b65673d
@tiagofilipe12
fixed pipeline and removed input which was causing issues
f8abcef
@tiagofilipe12
passed some more options to redux logger
297e02f
@tiagofilipe12
solved bug that prevented to log to graphson the resolvedInput
9620ab7
@tiagofilipe12
Cleaned some unused code.
Loading status checks…
b198bba

@tiagofilipe12 tiagofilipe12 added enhancement feature labels Jul 7, 2017

@tiagofilipe12 tiagofilipe12 self-assigned this Jul 7, 2017

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

@tiagofilipe12 tiagofilipe12 referenced this pull request Jul 7, 2017

Closed

Summary file #57

lib/reducers/collection.js
return obj
}
// writes each graphson to file
fs.writeFileSync('graphson.json', JSON.stringify(graphsonObj, null, 2))

This comment has been minimized.

Sign in to view
Copy link
@thejmazz

thejmazz Jul 8, 2017

Member

should be async in future

@thejmazz thejmazz merged commit 8d227e5 into master Jul 8, 2017
0 of 3 checks passed

0 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build failed
Details
security/snyk 1 new vulnerable dependency path
Details

@tiagofilipe12 tiagofilipe12 deleted the viz branch Jul 8, 2017

@bmpvieira bmpvieira removed the enhancement label Aug 22, 2017

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