Skip to content

/ media

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

Use new graph stuff! #47

Merged
merged 8 commits into from Jun 8, 2018
Merged

Use new graph stuff! #47

merged 8 commits into from Jun 8, 2018
+124 −37

Conversation

@Manishearth
Copy link
Member

Manishearth commented Jun 7, 2018 •
edited

This uses the GraphImpl I wrote in #45

Seems to be working well.
Manishearth added 7 commits Jun 7, 2018
@Manishearth
Make render thread use GraphImpl
94408cf
@Manishearth
Make node/port ids structs
43f4c92
@Manishearth
Convenience methods for crafting port ids
3e6d93e
@Manishearth
Ability to connect nodes
ba739f7
@Manishearth
Fix bug in graph impl
4db63ec
@Manishearth
Fix destination node's output count
9574cbd
@Manishearth
Use new graph in play example
6509bc9
@Manishearth Manishearth requested a review from ferjm Jun 7, 2018
@Manishearth
Document graph impl
7f9ab87
@ferjm
ferjm approved these changes Jun 8, 2018
View changes
Copy link
Member

ferjm left a comment

\o/
@ferjm ferjm merged commit 540d049 into master Jun 8, 2018
@ferjm ferjm deleted the graph-use branch Jun 8, 2018
@ferjm ferjm mentioned this pull request Jun 8, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ferjm ferjm

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@Manishearth @ferjm
You can’t perform that action at this time.