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

Vector from direction #5502

Merged
merged 16 commits into from
Jun 28, 2023
Merged

Conversation

sovdeeth
Copy link
Member

@sovdeeth sovdeeth commented Mar 9, 2023

Description

Adds a syntax to create vectors from directions: vector[s] [from|in] [direction] %directions%

This allows things like:

vector in direction of player
vectors from north, south, east, and west

Relative directions will just be relative to the origin, with yaw/pitch 0.
I especially find the first example a useful replacement for the more unwieldy vector from yaw player's yaw and pitch player's pitch.

However, directions scare me so I'd love confirmation that this won't mess anything up, especially with how loose the syntax is.


Target Minecraft Versions:
Requirements:
Related Issues:

@AyhamAl-Ali AyhamAl-Ali added the feature Pull request adding a new feature. label Mar 9, 2023
Copy link
Member

@AyhamAl-Ali AyhamAl-Ali left a comment

Choose a reason for hiding this comment

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

Nice PR ⚡

From discussion with Pikachu in SkUnity discord, for allowing `northwards vector` and similar.
Copy link
Member

@AyhamAl-Ali AyhamAl-Ali left a comment

Choose a reason for hiding this comment

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

Looking good, just couple changes

@sovdeeth sovdeeth requested a review from TheLimeGlass May 2, 2023 04:52
@TheLimeGlass TheLimeGlass merged commit f6e4ccf into SkriptLang:master Jun 28, 2023
4 checks passed
Moderocky pushed a commit to Moderocky/Skript that referenced this pull request Sep 16, 2023
@sovdeeth sovdeeth deleted the VectorFromDirection branch September 26, 2023 17:15
NotSoDelayed pushed a commit to NotSoDelayed/Skript that referenced this pull request Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Pull request adding a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants