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

impossible to change a Table entry #7382

Open
1 of 17 tasks
Cdriko opened this issue Nov 20, 2024 · 5 comments
Open
1 of 17 tasks

impossible to change a Table entry #7382

Cdriko opened this issue Nov 20, 2024 · 5 comments

Comments

@Cdriko
Copy link

Cdriko commented Nov 20, 2024

Most appropriate sub-area of p5.js?

  • Accessibility
  • Color
  • Core/Environment/Rendering
  • Data
  • DOM
  • Events
  • Image
  • IO
  • Math
  • Typography
  • Utilities
  • WebGL
  • Build process
  • Unit testing
  • Internationalization
  • Friendly errors
  • Other (specify if possible)

p5.js version

v1.9.4

Web browser and version

131.0

Operating system

linux

Steps to reproduce this

Steps:

  1. got to https://editor.p5js.org/Cdriko/sketches/rbr77d-AU
  2. click on "Deplace" button
  3. this execute the 'deplaceBloc' function in that The function tableRow.setString( give an error
  4. error 'Column #0 is out of the range of this table"
  5. But I read the row and the colomn just before.

Snippet:

@Cdriko Cdriko added the Bug label Nov 20, 2024
Copy link

welcome bot commented Nov 20, 2024

Welcome! 👋 Thanks for opening your first issue here! And to ensure the community is able to respond to your issue, please make sure to fill out the inputs in the issue forms. Thank you!

@Deathnote45
Copy link

hi, Can I Contribute on that issue? @Cdriko

@Cdriko
Copy link
Author

Cdriko commented Nov 29, 2024

hello @Deathnote45 you'r welcome ;)

@Deathnote45
Copy link

Can I contribute? @Cdriko

@stampyzfanz
Copy link
Contributor

stampyzfanz commented Dec 21, 2024

Hi @Deathnote45, it would might be helpful for you to help 'triage' this issue, and you don't need to ask before doing so if no one else has messaged yet. Its probably good to check before creating a pull request though just to make sure that your idea is on the right track and other people know that you're working on it.

For this issue, the sketch provided is 580 lines of code, which makes it difficult to work out where the issue is. It would be helpful for you to come up with the 'minimal reproducible example' where you share with us a snippet of code which is as small as possible but shows the issue. I asked ChatGPT to create a simple example from the code which you can find here. This code works correctly though, so ChatGPT didn't do this correctly, however, its a good starting place for what a simple code snippet showing the error could look like.

When doing so, you may find out that there isn't an error in the p5.js library, but an error is instead found in Cdriko's code, in which case you could help solve the issue and recommend we close the issue. In the future, it would be helpful if Cdriko made a small sketch that showed the issue without other code that may draw our attention away from the bug.

Edit: Sorry for my confidence, maintainers are actually talking about retiring p5.Table in #7090, so it may not be important to fix this issue if we remove the class.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants