Skip to content

Update maze viz #179

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

Closed
wants to merge 12 commits into from
Closed

Update maze viz #179

wants to merge 12 commits into from

Conversation

nadr0
Copy link
Contributor

@nadr0 nadr0 commented Jun 15, 2016

  • cleaned up some code
  • added more visuals to the array2Dtracer
  • added logger information to see what the code is doing
  • added random start and end locations

Also I don't know much about the merging process but it looks like there are a ton of other files in this merge or something (I see like 12 commits below)? I only edited my code inside the create_maze. I'll see if this pull request works?

Yeah I am rather confused about the other commits. I thought I grabbed the most recent repo.

Oh, I think I used the master branch and not the gh-pages one? If it is too much of a hassle to sort this out you can just cancel the pull request and I can just resubmit one. I have my code saved.

@64json
Copy link
Member

64json commented Jun 15, 2016

We are now deploying Algorithm Visualizer to Heroku (PaaS) instead of GitHub Pages, and it seems I was working on it after you forked.
I will take care of this pull request tomorrow, and you may not need to worry about it 😀

@64json
Copy link
Member

64json commented Jun 16, 2016

merged in 9c2f3d2

@64json 64json closed this Jun 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants