Rectangle function map missing trimmed

[aaronventure]

Referring to any property will return the last one referred, so if you refer to rect.x and then to rect.wtf, the second one will return rect.x

Also, not having .centre or getCentre() or .scaledCentred(scalingX, scalingY) is also a bummer, seems like really low hanging fruit.

[d.healey]

@aaronventure said in Rectangle function map missing trimmed:

Referring to any property will return the last one referred, so if you refer to rect.x and then to rect.wtf, the second one will return rect.x

Not sure what you mean here, got a snippet?

@aaronventure said in Rectangle function map missing trimmed:

Also, not having .centre or getCentre() or .scaledCentred(scalingX, scalingY) is also a bummer, seems like really low hanging fruit.

We have .withCentre and .reduced

[aaronventure]

@d-healey said in Rectangle function map missing trimmed:

Not sure what you mean here, got a snippet?

Call .x in a print command, then change the property to .whatever and hit F5 again.

@d-healey said in Rectangle function map missing trimmed:

We have .withCentre and .reduced

Alright I actually managed to add scaledCentre to the source, it works but it's not showing up in the API or autocomplete. How do I do that before making a PR?

[d.healey]

@aaronventure said in Rectangle function map missing trimmed:

Call .x in a print command, then change the property to .whatever and hit F5 again.

I see, so you if you use a non-existent key it returns the value from the last valid key you used.

@aaronventure said in Rectangle function map missing trimmed:

Alright I actually managed to add scaledCentre to the source,

What is the difference between this and reduced()?

Functions we add don't show up in the API because Christoph has to run some magic batch script for that.

[aaronventure]

@d-healey said in Rectangle function map missing trimmed:

What is the difference between this and reduced()?

Reduced trims half what you put in from each side, scaledCentre allows you to put in a scaling factor while keeping the rectangle centred. You could technically do it with reduced by writing a custom function of course, i mean you can do with scaled by adding a translation, but you can say that for half of the methods in the class.

@d-healey said in Rectangle function map missing trimmed:

Functions we add don't show up in the API because Christoph has to run some magic batch script for that.

Interesting. Alright, then!

[d.healey]

@aaronventure said in Rectangle function map missing trimmed:

scaledCentre allows you to put in a scaling factor while keeping the rectangle centred.

Isn't that the same as .scaled()?

Oh I see, scaled only works from the top left corner.

[aaronventure]

@d-healey yeah, there's withSizeKeepingCentre but that still requires fetching width and height first

Anyway, PR's up

- add centre-related methods to the Rectangle class by aaronventure · Pull Request #717 · christophhart/HISE

Added getCentre(), getCentreX() and getCentreY() from the base Rectangle class Added scaledCentre() which will perform factor scaling while keeping the same centre Removed that unused p because why...

GitHub (github.com)

[d.healey]

@d-healey said in Rectangle function map missing trimmed:

We seem to be missing toArray, I tried to add this myself but kept getting some weird errors that I don't understand.

Solved

[d.healey]

Looks like .setCentre doesn't work, and why are there two setSize functions?

[d.healey]

When using g.addNoise() the area parameter is still expecting an array.