-
Notifications
You must be signed in to change notification settings - Fork 518
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
MiniCat-Armaanpreet #644
MiniCat-Armaanpreet #644
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Thanks for the submission. Could you merge some of the circles together? |
yes |
yes
…On Tue, Jul 9, 2024 at 3:22 AM leomcelroy ***@***.***> wrote:
Thanks for the submission. Could you merge some of the circles together?
—
Reply to this email directly, view it on GitHub
<#644 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/BJXG7V3GOK7T5JJON6M4G3TZLM3MZAVCNFSM6AAAAABKQV2J6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJWGAYTAMJXGE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
i changed index.js |
Hi, how long it takes to approve my drawing
…On Tue, Jul 9, 2024 at 3:22 AM leomcelroy ***@***.***> wrote:
Thanks for the submission. Could you merge some of the circles together?
—
Reply to this email directly, view it on GitHub
<#644 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/BJXG7V3GOK7T5JJON6M4G3TZLM3MZAVCNFSM6AAAAABKQV2J6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJWGAYTAMJXGE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @4n0nym0u5-3y3– this is a good start but I don't think it meets the requirement for being generative! Could you play with this a bit more and make it generate a new cat every time so I can merge your PR?
Ok |
i changed the script |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure to rename your file index,js.js
-> index.js
and update your screenshots!
Hi, i updated my file from index,js.js -> index.js |
Nice work @4n0nym0u5-3y3, now update the screenshots so i can see your new generative code! |
I'm having problems to update snapshots what can i do? |
Post in the slack if you need help getting the PR updated! Maybe someone there can help. |
snapshots updated |
Hi, updated snapshots and my file from index,js.js -> index.js |
|
Also just an FYI, using math.random() instead of bt.randInRange(min: number, max: number) may make it so what is on the preview and whats drawn by the blot different |
Thanks for the information |
So the code is fine as it is |
It would be better to change it to use the toolkit function(I think it might even be a requirement now) |
i updated my code |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is not good the project |
So it's approved my project |
?? |
i have to update something |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, I like the idea and aesthetic of the design, but I feel as though you can polish this even more - could you keep working on this?
i updated my code |
i added somethigs to make it cute |
like bell |
. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the improvements! I think you could still polish this piece more though. Is there a way you can remove all the overlapping lines?
i will try |
@4n0nym0u5-3y3 is attempting to deploy a commit to the Hack Club Team on Vercel. A member of the Team first needs to authorize it. |
i tried but nothing |
Hi there! I’m closing this PR for now to clear up the repo. Once you’ve made your edits/improvements, please make a new PR and we can review it there!! |
My artwork is a digital representation of a Mini Cat created through coding. Using JavaScript and simple geometric shapes defined by mathematical formulas, I've constructed a charming depiction of Mini Cat with distinct features such as its round head, ears, eyes, nose, mouth, and paws. Each element, from the smooth curves of its head to the circular shapes representing its eyes and paws, is meticulously drawn using basic drawing primitives and mathematical calculations.
Coding this artwork was an engaging experience that combined creativity with technical skill. It involved planning out the composition using coordinates and dimensions, implementing algorithms to draw circles and lines, and iterating to refine details like shading and proportion. Each part of the Mini Cat, from its ears to its hands, required careful consideration of positioning and size to ensure the overall balance and appeal of the artwork.
I enjoyed the challenge of translating artistic vision into code. It allowed me to explore the intersection of art and programming, leveraging logic and creativity simultaneously. Adjusting parameters such as segment counts for smoother curves or adding subtle shading for depth provided opportunities to refine and enhance the visual impact of the Mini Cat illustration.
Overall, coding this artwork was not only about creating a visual representation but also about the journey of problem-solving and iterative improvement. It underscored the versatility of coding as a medium for artistic expression, where precise calculations and algorithms converge to produce visually compelling results.