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.

Sign up for GitHub

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

Jump to bottom

Added parsing and output of dependency graph #54

Merged
merged 5 commits into from
Mar 22, 2016
Merged

Added parsing and output of dependency graph #54

merged 5 commits into from
Mar 22, 2016

Conversation

mgeplf
Copy link
Contributor

@mgeplf mgeplf commented Mar 22, 2016

@mgeplf
Added parsing and output of dependency graph
b953b92 
* parses Dependency.txt style files
* Outputs graphviz, or flat text
* as suggested by feature:
    BlueBrain#6
@mgeplf mgeplf mentioned this pull request Mar 22, 2016
Open
@codecov-io
Copy link

Current coverage is 100.00%

Merging #54 into master will not affect coverage as of e66cb3d

@@            master     #54   diff @@
======================================
  Files            5       4     -1
  Stmts          151     118    -33
  Branches         0       0       
  Methods          0               
======================================
- Hit            151     118    -33
  Partial          0       0       
  Missed           0       0

Review entire Coverage Diff as of e66cb3d

Powered by Codecov. Updated on successful CI builds.

mgeplf and others added 2 commits March 22, 2016 10:00
@mgeplf
Merge branch 'master' into graph_dependencies
ce35b06
@mgeplf
Added parsing and output of dependency graph
0e0eb98 
* parses Dependency.txt style files
* Outputs graphviz, or flat text
* as suggested by feature:
    BlueBrain#6
@wvangeit
Copy link
Contributor

Can you add a target to the Makefile to build this ?
Maybe we commit an example so that it can be visualised on github ?

@mgeplf
Copy link
Contributor Author

mgeplf commented Mar 22, 2016

Ok.

The diagram is huge, fyi.

@wvangeit
Copy link
Contributor

You mean huge in file size ? Can it be rasterized ?

@mgeplf
Copy link
Contributor Author

mgeplf commented Mar 22, 2016

I added a target for the Makefile.

I don't think we should upload the graph yet. It probably needs a bit more love by someone who is more talented with graphviz, and can have it layout better.

@wvangeit
Copy link
Contributor

Ok, understood, thanks for the Makefile target

wvangeit added a commit that referenced this pull request Mar 22, 2016
@wvangeit
Merge pull request #54 from mgeplf/graph_dependencies
0d2d835 
Added parsing and output of dependency graph
@wvangeit wvangeit merged commit 0d2d835 into BlueBrain:master Mar 22, 2016
@wvangeit
Copy link
Contributor

Cool graph ... :-)

ilkilic pushed a commit that referenced this pull request Dec 8, 2023
@DrTaDa
Merge pull request #54 from BlueBrain/minor_changes
d6482d6 
Minor changes to eCode detection and fix double extraction of some features
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mgeplf @codecov-io @wvangeit