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

Wanted Posters #932

Closed
wants to merge 2 commits into from
Closed

Wanted Posters #932

wants to merge 2 commits into from

Conversation

grandell1234
Copy link

My Blot creation is wanted posters. I spent a long time working on them because I created a "custom text engine" function so I could call (an example from my script): drawText('WANTED', 50 * scaleX, 574 * scaleY, 63 * scaleX, 5 * scaleX);
Every time you run the program a different wanted guy will appear with different facial features. It took about two months to get this completed, NOTE: A few weeks was spent planning what I wanted to make.

I have plans for a future project that I think could be kind of fun for Club organizers, but after some playing around in the blot editor, I found that I really need an actual machine I can proto-type on to continue my work on that.

I would definitely recommend running the script yourself in the Editor, to see a bunch of the different faces you can get (as its randomized everytime).

Thank you for spending time reviewing my submission!

I also want to note, that for the last thing you are marking off, "colour" is misspelled and should be "color" and in the line talked about if you used a tutorial or remix, there is an extra "used" before "based".

  • I have read the steps to getting a blot

  • I have read all the submission requirements and follow them

  • I am submitting art that...

    • is algorithmically generated (will meaningfully change each time the program is run)
    • is your own work that is not copied from somewhere else
    • is drawable through the editor(does not use outside tools)
    • doesn't call Math.random() (See the documentation on randomness)
    • is drawable on Blot itself (doesn't have lines overlap more than 5 times)
      • Doesn't overlap lines more than 5 times
      • Fits in the working area of 125x125mm
      • Doesn't rely on colour to look good
  • If you used based your art on something else, used a tutorial, or remixed it from something else, please link it here:

Lastly, link a picture of your art piece here:
WantedPoster3

Copy link

vercel bot commented Oct 2, 2024

@grandell1234 is attempting to deploy a commit to the Hack Club Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

github-actions bot commented Oct 2, 2024

art/WantedPoster/index.js looks like art! preview it in the editor

@Dongathan-Jong
Copy link
Collaborator

Hey! I love the idea, but this piece falls under the bar due to simplicity. I would recommend making it so that it could change names and have a part on the bottom that says "REWARD: $XXX" with a different number each time! I also suggest you use BlotFont, an easier way to make art! Once you update your art I will be happy to re-review it!

@Dongathan-Jong Dongathan-Jong self-assigned this Oct 3, 2024
@grandell1234
Copy link
Author

Thank you, I will make the specified changes.

@grandell1234
Copy link
Author

Hello @Dongathan-Jong, before I do a resubmit is this more what you are looking for. There is now a reward that changes every time, along with a name that changes every time. To make sure it always fits in the required spacing I have every first name the same length, along with the last name (5-7 characters):

const firstNames = ['JAMES', 'SARAH', 'DAVID', 'CHRIS', 'ALICE', 'EMILY', 'BRIAN', 'OLIVE'];
const lastNames = ['JOHNSON', 'MARTINE', 'GARCIA', 'WILLIAM', 'BROWNIE', 'JACKSON', 'MILLARD', 'ROBISON'];

Screenshot 2024-10-04 at 10 48 24 AM

@Dongathan-Jong
Copy link
Collaborator

Again, these are still pre-generated! I want to see something that has infinite possibilities, not random with a set number of cases.

@M0HID
Copy link
Collaborator

M0HID commented Oct 5, 2024

Hi there! I’m closing this PR for now to clear up the repo. As said by Dongathan-Jong, please make the reward value completely randomised! Once you’ve made your edits/improvements, please make a new PR and we can review it there!!

@M0HID M0HID closed this Oct 5, 2024
@grandell1234
Copy link
Author

Yes, but can I put a max on the reward (otherwise it just goes off-screen, and doesn't meet that requirement.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants