Skript icon indicating copy to clipboard operation
Skript copied to clipboard

Vector function that accepts a single argument

Open miberss opened this issue 1 month ago • 12 comments

Suggestion

I'd like to suggest a simple overload on vector to allow for single number arguments to be accepted where, vector(n) = vector(n, n, n).

Why?

This is useful for developers who require all-n vector easier.

A possible use-case of a all-n vector would be display scale of a display.

I find myself writing this function commonly and it would be nice to have it in the base of Skript. It is also included in many other vector libraries as a common overload.

Other

No response

Agreement

  • [x] I have read the guidelines above and affirm I am following them with this suggestion.

miberss avatar Oct 30 '25 16:10 miberss

The take on this around 2 weeks ago was that it would be too ambiguous, but maybe it can be discussed again?

erenkarakal avatar Oct 30 '25 16:10 erenkarakal

The take on this around 2 weeks ago was that it would be too ambiguous, but maybe it can be discussed again?

Where was it discussed?

miberss avatar Oct 30 '25 16:10 miberss

Where was it discussed?

private

erenkarakal avatar Oct 30 '25 17:10 erenkarakal

Screenshot_20251030-220219.png

Wasn't much discussion I still think it's rather ambiguous and easy for a user to make if they want it, but I can be convinced if it's popular

sovdeeth avatar Oct 31 '25 02:10 sovdeeth

I don't personally see a problem with it

UnderscoreTud avatar Oct 31 '25 05:10 UnderscoreTud

can i take up this issue i want to contribute its looks interesting

harshasiddartha avatar Oct 31 '25 18:10 harshasiddartha

can i take up this issue i want to contribute its looks interesting

sure

erenkarakal avatar Oct 31 '25 18:10 erenkarakal

@erenkarakal i have fixed the issue . can you review and merge

harshasiddartha avatar Oct 31 '25 19:10 harshasiddartha

@erenkarakal i have fixed the issue . can you review and merge

He's not the only one that can review, other people on skriptlang can, and it's not going to be merged in a day. There's a system in place that the members of the team have to follow to merge changes, like reviewing the code, testing it, and so on.

EquipableMC avatar Oct 31 '25 20:10 EquipableMC

@EquipableMC yeah sure you guys can review and test and close the issue

harshasiddartha avatar Oct 31 '25 20:10 harshasiddartha

@EquipableMC yeah sure you guys can review and test and close the issue

They will when they can

EquipableMC avatar Oct 31 '25 20:10 EquipableMC

We keep issues open until release, which will be in January for this feature

sovdeeth avatar Oct 31 '25 20:10 sovdeeth