P3D crash with serialEvent

[MazeArt]

Hey Guys,
I am trying to draw a 3d object based on readings of a gyroscope sensor.
I get a Fatal Error :

#A fatal error has been detected by the Java Runtime Environment:
#SIGSEGV (0xb) at pc=0x00007fff7ac58e31, pid=27319, tid=0x0000000000012a03`
#
#JRE version: Java(TM) SE Runtime Environment (8.0_144-b01) (build 1.8.0_144-b01)
#Java VM: Java HotSpot(
#TM) 64-Bit Server VM (25.144-b01 mixed mode bsd-amd64 compressed oops)
# Problematic frame:
# C  [libGL.dylib+0xe31]  glClearColor+0xd
#
# Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /Users/Delia/hs_err_pid27319.log
#
# If you would like to submit a bug report, please visit:
#   http://bugreport.java.com/bugreport/crash.jsp
# The crash happened outside the Java Virtual Machine in native code.
# See problematic frame for where to report the bug.
#
Could not run the sketch (Target VM failed to initialize).
For more information, read revisions.txt and Help → Troubleshooting.

This is the Code :

//https://forum.processing.org/one/topic/rotate-cube-sensor.html 

import processing.serial.*;     // import the Processing serial library
Serial myPort;                  // The serial port

float bgcolor;            // Background color
float fgcolor;            // Fill color
float xpos, ypos;            // Starting position of the ball

void setup() {
  frameRate(30);
  size(640,480, P3D);
  background(0);
  lights();
  // List all the available serial ports
 // println(Serial.list());
  myPort = new Serial(this, "/dev/cu.usbmodem1421", 38400);

  // read bytes into a buffer until you get a linefeed (ASCII 10):
  myPort.bufferUntil('\n');
  
  // draw with smooth edges:
  smooth();
}

void draw() {
}

void serialEvent(Serial myPort) { 
  background(0);
  noStroke();
  pushMatrix();
  translate(width/2, height/2);
  
  // read the serial buffer:
  String myString = myPort.readStringUntil('\n');
  // if you got any bytes other than the linefeed:
    myString = trim(myString);
 
    // split the string at the commas
    // and convert the sections into integers:
    float sensors[] = float(split(myString, ','));
//println("roll: " + sensors[0] + " pitch: " + sensors[1] + " yaw: " + sensors[2] + "\t");
    // print out the values you got:
    for (int sensorNum = 0; sensorNum < sensors.length; sensorNum++) {
      print("Sensor " + sensorNum + ": " + sensors[sensorNum] + "\t");
     if (sensorNum == 0) {
       println("Roll");
       //rotateX(sensors[0]);

     }
     if (sensorNum == 1) {
       println("Pitch");
       //rotateY(sensors[1]);

     }
     if (sensorNum == 2) {
       println("Yaw");
       //rotateZ(sensors[2]);
     }
     
    }
    // add a linefeed after all the sensor values are printed:
    println();
    box(100);
    popMatrix();
    if (sensors.length > 1) {
      rotateX(sensors[0]);
      rotateY(sensors[1]);
      rotateZ(sensors[2]);
    }
    // send a byte to ask for more data:
    myPort.write("A");
  }

##Environment

• Processing version: 3.3.6
• Operating System and OS version: OS Sierra 10.12.6
  *Macbook Pro Retina, 15-inch, Mid 2014;
  *Graphics Card : NVIDIA GeForce GT 750M 2048 MB
  Intel Iris Pro 1536 MB

Any help would be greatly appreciated, thanks!

[gohai]

I am pretty sure the crash is due to the fact that your code is drawing to the screen inside serialEvent, which can be invoked at any time - including during periods in the draw-loop where Processing is not supposed to emit draw operations.

We discussed this pitfall when the Serial library was re-written, see here.

@MazeArt Could you try changing serialEvent to (the undocumented) serialAvailable, and check if this works? (If it does, then this was indeed the issue, and you might want to then move all your code inside draw and make use of available(), instead of serialEvent).

[MazeArt]

Thank you @gohai for your response.
I am not very experienced ...So I can barely understand the solution.
I replaced literally void serialEvent(Serial myPort) {... with void serialAvailable(Serial myPort) {...
This now shows me this error:

Error, disabling serialAvailable() for /dev/cu.usbmodem1421
null

i will now try to move everything inside draw ....to see if it works.

[MazeArt]

I changed the whole code and now it is working!
The problem now is that a few times running the code, it shows the error:

Error, disabling serialAvailable() for /dev/cu.usbmodem1421
null

Having to disconnect and reconnect the arduino every few runs....
Any idea why this might be happening??
This is the new code:

float xx,yy,zz;
import processing.serial.*;     // import the Processing serial library
Serial myPort;                  // The serial port

float bgcolor;            // Background color
float fgcolor;            // Fill color
float xpos, ypos;            // Starting position of the ball

void setup() {
  size(640,480, P3D);

  // List all the available serial ports
  //println(Serial.list());

 myPort = new Serial(this, "/dev/cu.usbmodem1421", 38400);

  // read bytes into a buffer until you get a linefeed (ASCII 10):
  myPort.bufferUntil('\n');
  
  // draw with smooth edges:
  smooth();
}

void draw() {
  background(0);
  noStroke();
  translate(width/2, height/2);
  pushMatrix();
  rotateX(yy);//pitch
  rotateY(zz);//yaw
  rotateZ(xx);//roll
  box(50, 20, 300);
  popMatrix();
 // println("roll: " + xx);
 // println("pitch: " + yy);
 // println("Yaw: " + zz);
}


void serialEvent(Serial myPort) { 
  // read the serial buffer:
  String myString = myPort.readStringUntil('\n');
  // if you got any bytes other than the linefeed:
    myString = trim(myString);
   //println(myString); //to see what's happening
 
    // Identify the 
    // and convert the sections into integers:
  if(myString.substring(0,4).equals("roll")==true){
   println(myString);
   int stringLen = myString.length();
   println(stringLen);
   
   stringLen = myString.indexOf(" ");
   println(stringLen);
        myString = myString.substring(stringLen);
        println(myString);
        float sensors[] = float(split(myString, ','));
        
    
//println("roll: " + sensors[0] + " pitch: " + sensors[1] + " yaw: " + sensors[2] + "\t");
    // print out the values you got:
    for (int sensorNum = 0; sensorNum < sensors.length; sensorNum++) {
      //print("Sensor " + sensorNum + ": " + sensors[sensorNum] + "\t"); 
      if (sensorNum == 0) {
        xx = sensors[0];
      }
      if (sensorNum == 1) {
        yy = sensors[1];
      }
      if (sensorNum == 2) {
        zz = sensors[2];
      }
    }
    }
    // add a linefeed after all the sensor values are printed:
    //println();

    // send a byte to ask for more data:
    myPort.write("A");
  }

[gohai]

@MazeArt What happens if you get rid of the serialEvent?

Instead, put it all inside draw like so:

void draw() {
  if (myPort.available() > 0) {
    // ...
  }
}

[MazeArt]

It worked !! Thank you

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.