processing-website icon indicating copy to clipboard operation
processing-website copied to clipboard

Reference documentation, problem in code example given for PImage.height

Open MegaCurious opened this issue 1 year ago • 3 comments

Select the type of content error.

Factual Mistake

Section

Reference

Location of the error

content/references/examples/processing/PImage_height/PImage_height.pde

Describe the error

The code example given in reference section for Pimage_height doesn’t have any mention at all of height in it. It should be updated to be relevant to the heading under which it occurs.

Suggested correction

Perhaps the content could be updated to be parallel in structure to the documentation for Pimage_width

Language

English

Screenshots or references

// this is the code example given, which does not illustrate how PImage.height works and does not even reference it

PImage photo;

void setup() { size(400, 400); photo = loadImage("mt-horai.jpg"); }

void draw() { image(photo, 0, 0); }

Additional context

No response

MegaCurious avatar Jan 23 '25 13:01 MegaCurious

Hi @MegaCurious and thanks for reporting this issue. I see this is your first one on GitHub. Congratulations and welcome!

For reference, the file with the code for this example is located here:

/content/references/examples/processing/PImage_height/PImage_height.pde

Would you like to help resolve this issue? I'll be happy to offer any guidance on making your first contribution.

SableRaf avatar Jan 23 '25 14:01 SableRaf

Thanks for the reply and invite, SableRalph!

I'm down a rabbit hole at the moment but if nobody has gotten to it by the time I come back I would be happy to give it a go - I'll let you know.

  • Heather

On Thu, Jan 23, 2025 at 2:03 PM Raphaël de Courville < @.***> wrote:

Hi @MegaCurious https://github.com/MegaCurious and thanks for reporting this issue. I see this is your first one on GitHub. Congratulations and welcome!

For reference, the file with the code for this example is located here:

/content/references/examples/processing/PImage_height/PImage_height.pde https://github.com/processing/processing-website/blob/main/content/references/examples/processing/PImage_height/PImage_height.pde

Would you like to help resolve this issue? I'll be happy to offer any guidance on making your first contribution.

— Reply to this email directly, view it on GitHub https://github.com/processing/processing-website/issues/605#issuecomment-2609890247, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACTEPQP3DPXU4JXNUNES5Q32MDZC5AVCNFSM6AAAAABVXKQLISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMBZHA4TAMRUG4 . You are receiving this because you were mentioned.Message ID: @.***>

MegaCurious avatar Jan 28 '25 13:01 MegaCurious

Sorry - just saw it's SableRaph not SableRalph

atb

Heather

On Tue, Jan 28, 2025 at 1:50 PM Heather Stark < @.***> wrote:

Thanks for the reply and invite, SableRalph!

I'm down a rabbit hole at the moment but if nobody has gotten to it by the time I come back I would be happy to give it a go - I'll let you know.

  • Heather

On Thu, Jan 23, 2025 at 2:03 PM Raphaël de Courville < @.***> wrote:

Hi @MegaCurious https://github.com/MegaCurious and thanks for reporting this issue. I see this is your first one on GitHub. Congratulations and welcome!

For reference, the file with the code for this example is located here:

/content/references/examples/processing/PImage_height/PImage_height.pde https://github.com/processing/processing-website/blob/main/content/references/examples/processing/PImage_height/PImage_height.pde

Would you like to help resolve this issue? I'll be happy to offer any guidance on making your first contribution.

— Reply to this email directly, view it on GitHub https://github.com/processing/processing-website/issues/605#issuecomment-2609890247, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACTEPQP3DPXU4JXNUNES5Q32MDZC5AVCNFSM6AAAAABVXKQLISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMBZHA4TAMRUG4 . You are receiving this because you were mentioned.Message ID: @.***>

MegaCurious avatar Feb 18 '25 12:02 MegaCurious