-
Notifications
You must be signed in to change notification settings - Fork 16
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
OpenGL Teapot the return #30
Comments
@darnuria This was one of the many projects suggested by @noalien during our email exchanges so i guess it's ok. I've already started working on it. Finding the original The main question i'm asking myself being how do i connect myself to this API. @noalien, is it better to create a whole new file dedicated to those two functions somewhere in |
Oh, please make it online as a fork and open a PR would make review easier for us! ^_^ Don't mind about making mistakes, changing mind or whatever every human being is imperfect it's OK. If you did'nt plan to do malicious stuff. ;p Moreover: we are here to help. ;) |
Great idea @darnuria. I'm not doing it today, but i sure will. Probably next week. I'll keep you informed. |
I added a document that specifies what is expected for gl4duTeapot : https://github.com/noalien/GL4Dummies/blob/master/docs/TODO_gl4dgTeapot.pdf |
BEHOLD THE TEAPOTteapot.mp4 |
Good start |
It was actually just a lack of a Teapot.1.mp4I'm kind of proud actually. I had to work basically blindly for three weeks. I could'nt see what i was doing and still managed to be one |
Sadly i've gone through multiple unexpected tribulations. I dont want to complain to much but the task wasn't easy. Everything seemed to work against me. I encountered multiple problems even from official models and i've grown tired of it. That was really upsetting seeing one week worth of work getting thrown because a well known library actually contained misaligned vertices. Knowing this and the fact that i have to work on other projects, I won't be able to continue adding the spout/handle. This projet was meant to target multiple people so i guess it's normal, but I prefer to prevent. If someone wants to add it and has the guts to add hundreds of vertices, i have the perfect model i just made myself to do the work. Adding it with some tweaks to the texCoord should be enough. Just ask and i will deliver ! I'm even willing to mentor a bit if needed ;) |
Keep in mind that ideally the teapot should take advantage of the "mkRegularGrid *" functions and use a mesh generator based on surfaces of revolution. Good luck. |
Sorry, but i'm busy these times and I won't be able to implement such a thing :/ I hardly understand the Moreover your pdf emphasizes on the fact that i should implement the teapot like the cylinder and that's what I think I did. Besides, my teapot generation is already quite optimized, as much in terms of algorithmic complexity as in terms of execution time. It only require one single I'll see if I can free myself some time at the beginning of next month, but it is more likely that I will work on other projects. I'll keep you informed if it happens :) |
@Marobax The message was not directly addressed to you, sorry :). I insisted for the "next one", who would take up the subject (to give the teapot an oriental look). |
Hello, we are three students who want to work on the Teapot, @Laozey @Bibi210 and myself. We have started to implement the main part of the Teapot and the cover based on the functions "mkSphereVerticesf" and "gl4dgGenSpheref". We obtain for the moment the result below, it seems to us that the calculations of the normals and the coordinates of textures must still be improved. Capture_decran_video_de_25-02-2022_140713.mp4We are currently implementing the use of Bézier curves in order to do the other parts of the Teapot or even to use them for what we have already done. For example, below, we place spheres on a Bézier curve defined by 6 points. |
Nice beginning |
@noalien shall we close it? (auto-triaging my own openned issues) |
unfortunately no, the specifications given above (#30 (comment)) are not satisfied.
|
@Marobax said 2020-02-04:
The text was updated successfully, but these errors were encountered: