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

Draft updates to use py5 in first python path #122

Draft
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

create-issue-branch[bot]
Copy link
Contributor

closes #118

@raspberrypiherokubot raspberrypiherokubot temporarily deployed to editor-api-p-issues-118-de3nuo February 3, 2023 15:32 Inactive
@chrisruk chrisruk force-pushed the issues/118-Draft_updates_to_use_py5_in_first_python_path branch from 6d11c68 to 5cb8ca7 Compare February 6, 2023 15:38
shoot_arrow()

# Keep this to run your code
run(frame_rate=2)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to add py5.frame_rate(2) at the start of setup to make up for this

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup, good spot, just done

py5.fill('sienna') # Set the arrow fill colour to brown
py5.circle(arrow_x, arrow_y, 15) # Draw a small circle at random coordinates

def settings():
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this can go back in setup now?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, just done

@chrisruk chrisruk force-pushed the issues/118-Draft_updates_to_use_py5_in_first_python_path branch from 3402c22 to 55098b4 Compare February 9, 2023 14:44
@chrisruk chrisruk force-pushed the issues/118-Draft_updates_to_use_py5_in_first_python_path branch from 4bb4ea5 to 65c1b13 Compare February 9, 2023 14:52
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.

4 participants