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.

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

Orientation #54

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Orientation #54

wants to merge 3 commits into from

Conversation

smitpatwa
Copy link

Fix for issue #52.

@hvk3
Copy link

hvk3 commented Mar 8, 2016

There seems to be some problem in your implementation, I guess. Try constructing a full binary tree of 7 nodes in vertical orientation, and view that in horizontal orientation.

@stengel
Copy link
Member

stengel commented Mar 8, 2016

Hi, this is correct - I do not think horizontal orientation
works at all yet.

Could you please introduce yourself so that your activity is
going in the right direction?

Thanks and regards,
--Bernhard

There seems to be some problem in your implementation, I guess. Try constructing a full binary tree of 7 nodes in vertical orientation, and view that in horizontal orientation.


Reply to this email directly or view it on GitHub:
#54 (comment)

Prof Bernhard von Stengel email: stengel@nash.lse.ac.uk
Department of Mathematics http://www.maths.lse.ac.uk/Personal/stengel
London School of Economics phone: +44-20-7955 6438 (office)
Houghton St, Room COL 4.12 +44-20-7226 2325 (home)
London WC2A 2AE, United Kingdom

@smitpatwa
Copy link
Author

@hvk3
I have checked many cases for vertical and corresponding horizontal orientation.
I cannot find any error.
Can you please share a screenshot where it is not drawing the tree properly?

@smitpatwa
Copy link
Author

screenshot 9
screenshot 10

This are the screen shot of a tree in horizontal orientation and vertical orientation

@smitpatwa
Copy link
Author

@stengel
I am Smit Patwa, pursuing BE(Hons) Computer Science at Birla Institute of Technology and Science, Pilani (BITS Pilani,India).
I am interested in working for this project. I got this project link from the GSOC 2016 website.

@hvk3
Copy link

hvk3 commented Mar 8, 2016

@smitpatwa
I believe I made some error while testing your code. Works fine now. Sorry for the inconvenience.

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