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

Triangle Shape implementation in Joy Library #22

Closed
kushalpokharel opened this issue Oct 6, 2023 · 13 comments
Closed

Triangle Shape implementation in Joy Library #22

kushalpokharel opened this issue Oct 6, 2023 · 13 comments

Comments

@kushalpokharel
Copy link
Contributor

Description

As of now, the Joy Library doesn't support drawing triangles. However, there is already an existing example of drawing a triangle.

  • A Triangle constructor would be added to the Shape type
  • The function to draw a triangle would accept three parameters of type Point.
  • The lines are drawn from the first point to the second, second to the third, and third to the first to make a triangle.

Future improvements

  • We could also add a function that draws a triangle by accepting two Points (A and B) an angle parameter (ABC) and the length of the side (BC) to draw the triangle
@joanita-51
Copy link
Contributor

Hi @kushalpokharel , are you looking for someone to work on this issue?

@Mankavelda
Copy link
Contributor

Description

As of now, the Joy Library doesn't support drawing triangles. However, there is already an existing example of drawing a triangle.

  • A Triangle constructor would be added to the Shape type
  • The function to draw a triangle would accept three parameters of type Point.
  • The lines are drawn from the first point to the second, second to the third, and third to the first to make a triangle.

Future improvements

  • We could also add a function that draws a triangle by accepting two Points (A and B) an angle parameter (ABC) and the length of the side (BC) to draw the triangle

Hello @kushalpokharel I can work on this issue

@TitilayoAdeyemi
Copy link
Contributor

Hi @Mankavelda Please I need help with setting up the ocaml-joy project on my system. I have been facing different errors and have not been able to contribute since. I'd really like for you to put me through on the steps to take. Thank you!

@joanita-51
Copy link
Contributor

@TitilayoAdeyemi , what operating system are you using?

@TitilayoAdeyemi
Copy link
Contributor

@TitilayoAdeyemi , what operating system are you using?

Hi @joanita-51 I am using a Windows operating system

@Sudha247
Copy link
Owner

Sudha247 commented Oct 8, 2023

Thanks for creating the issue. I discussed this offline with @nikochiko and we're not sure triangle is a good abstraction for joy. joy aims to have a really simple user facing API, and I'm not fully convinced a triangle would fit in well. However, a n-sided polygon with reasonable defaults might be a good addition. But, let's reserve that for later.

@Mankavelda
Copy link
Contributor

Mankavelda commented Oct 8, 2023

Hi @Mankavelda Please I need help with setting up the ocaml-joy project on my system. I have been facing different errors and have not been able to contribute since. I'd really like for you to put me through on the steps to take. Thank you!

Hello @TitilayoAdeyemi , did you follow this #1 ?
and please can you send a screenshot of the exact error you are facing? maybe I can know how to handle it from there

@kushalpokharel
Copy link
Contributor Author

Hi @Sudha247 , thank you for your feedback. We could discuss the n-sided polygon and how defaults will be chosen. I need to work on something. Please assign me a task

@joanita-51
Copy link
Contributor

@TitilayoAdeyemi , what operating system are you using?

Hi @joanita-51 I am using a Windows operating system

I think @nangahamandine would be more helpful, as she is using Windows. I'm using Ubuntu and followed the steps outlined in #1. You can also check #5 for details, where someone discussed the use of WSL for Windows users. and also look at #3 for the steps outlined by @AryanGodara

@nangahamandine
Copy link
Contributor

@TitilayoAdeyemi , what operating system are you using?

Hi @joanita-51 I am using a Windows operating system

I think @nangahamandine would be more helpful, as she is using Windows. I'm using Ubuntu and followed the steps outlined in #1. You can also check #5 for details, where someone discussed the use of WSL for Windows users. and also look at #3 for the steps outlined by @AryanGodara

Thanks for the suggestion @joanita-51. I've provided the suggestions that I could so far with installing the libraries on WSL, connecting WSL on VS Code and opening the project directory. Hopefully @TitilayoAdeyemi sorts it out after going through all the suggestions you've made.
Please get back to us if you still face issues after. If we can't help much then the mentors will certainly step in.

@TitilayoAdeyemi
Copy link
Contributor

Thank you so much @joanita-51
@nangahamandine Thank you so much too! I tried it out but it didn't work. I will delete and start over again, I'd give you feedback when I'm done with that. I am really grateful.

@nangahamandine
Copy link
Contributor

Thank you so much @joanita-51 @nangahamandine Thank you so much too! I tried it out but it didn't work. I will delete and start over again, I'd give you feedback when I'm done with that. I am really grateful.

You're welcome. I'm keeping my fingers crossed, hoping everything works. Will be here in case you have any further questions :)

@Sudha247
Copy link
Owner

Sudha247 commented Oct 9, 2023

@kushalpokharel I've created some new issues. Please feel free to take a look at them and pick up something.

@nikochiko nikochiko closed this as not planned Won't fix, can't repro, duplicate, stale Oct 10, 2023
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

No branches or pull requests

7 participants