processing / processing/processing4

Since v4.4.3 - XYscope has odd sizing bug and small examples alphabetical order issue

Đang mở
#1,225 1 bình luận 1 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

bug
Ngôn ngữ chính
Java
Star
494
Fork
183
Merge trung bình
4 giờ 39 phút
Pull request đã merge (30 ngày)
3

Mô tả

Most appropriate sub-area of Processing 4?

Core/Environment/Rendering

Processing version

4.4.3

Operating system

MacOS 13.6.5

Bug description

An issue was brought up on the XYscope repo about the size of shapes changing and going off screen.. did some version checking and discovered a bug was introduced in v4.4.3 of Processing. Prior to 4.4.2, my examples for 1_getting_started > calibration (or basic_drawing or clock) all worked fine, keeping the graphics within the 512x512 window. Since 4.4.3, the shapes seem to be 4x bigger... the sizing magnitude change is especially noticeable with the basic_drawing example where the mouseX/Y no longer match.. Odd that about half the sketches aren't affected.. so I'm guessing it's semi related to pushMatrix()/popMatrix() or translate()... really not sure.. Tried looking into the changelog, but perhaps someone working close on the project has an idea from the commit log what it might have been? The issue has lingered up until the latest 4.4.6 release.

v4.4.2, calibration example
Image

v4.4.3, calibration example (scale issue)
Image

On a small side note, another small issue that was introduced in 4.4.2, was the alphabetical order listing of the Contributed Libraries within the examples window. Previously XYscope was usually at the very bottom of this list, but now it looks like some other bits from the folder aren't being filtered? - aha, it's placing all lowercase libs at the bottom after any Titlecase ones...

v4.4.1, examples window
Image

v4.4.2, examples window (alphabetical issue)
Image

Steps to reproduce this
  1. Install XYscope library

  2. compare example calibration using v4.4.2 (or lower) and v4.4.3

snippet

/*
 calibration
 circle + square + lines to help center/adjust oscilloscpe display
 cc teddavis.org 2023
 */

import ddf.minim.*; // minim req to gen audio
import xyscope.*;   // import XYscope
XYscope xy;         // create XYscope instance

void setup() {
  size(512, 512);

  xy = new XYscope(this); // define XYscope instance
  //xy.getMixerInfo(); // lists all audio devices
}

void draw() {
  background(0);
  xy.clearWaves(); // clear waves similar to background

  pushMatrix();
  translate(width/2, height/2);
  pushMatrix();
  rotate(radians(180));
  xy.circle(0, 0, width);
  popMatrix();
  xy.rectMode(CENTER);
  xy.rect(0, 0, width);
  xy.line(-width/2, 0, width/2, 0);
  xy.line(0, height/2, 0, -height/2);
  popMatrix();

  xy.buildWaves(); // build audio from shapes
  xy.drawAll(); // draw all analytics
}
Additional context

No response

Would you like to work on the issue?

No, I’m just reporting the issue

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Bắt đầu bằng cách tái hiện các ví dụ XYscope calibration, basic_drawing và clock trên Processing 4.4.2 và 4.4.3, sau đó kiểm tra các thay đổi trong Core/Environment/Rendering ở phạm vi commit được liên kết. Kiểm tra xem regression về kích thước và regression về thứ tự của Contributed Libraries có các nguyên nhân riêng biệt hay không. Được xem là hoàn thành khi các ví dụ vẫn giữ nguyên kích thước mong đợi và danh sách thư viện được sắp xếp theo thứ tự bảng chữ cái một cách nhất quán.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
java
Lĩnh vực
computer-graphics, desktop
Loại issue
Lỗi
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
32/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.