Checklist for Status Label

[zbhujwala]

Errors/Status progress will be caught and updated into the "Status Bar" label

Statuses to account for:

• [x] Incorrect file type chosen (non-STL file type)
• [x] Implement status label
• [x] Printer not connected (can be generalized down to "Connection error")
• [x] G-code generation status (Working, Complete, Incomplete/Error)
• [x] Printer status (Working, Complete, Incomplete/Error, Paused, Stopped)
• [ ] Errors for Render (unsure about specifics, Rendering team is still working on their parts) -There may be other errors/statuses that I missed, please add them on to this issue so we can all work on it and keep it in mind.
• [ ] Push our final code to the class repo by Saturday

[peralta013]

Incorrect file type chosen (non-STL file type)

• Please review my Error Pop Up Window pull request.

[ZBoylan]

Steve mentioned we can use the serial.list() // Serial.list() // this will populate an array of strings of the ports on the user computer https://processing.org/reference/libraries/serial/Serial_list_.html

[ZBoylan]

The file name gets chopped off @peralta013 should we put it below the choose file button?

[ZBoylan]

"The createGraphics() function should almost never be used inside draw() because of the memory and time needed to set up the graphics. One-time or occasional use during draw() are acceptable, but code that calls createGraphics() at 60 frames per second will run out of memory or freeze your sketch." https://processing.org/reference/createGraphics_.html

Line 88 we do this

[GeorgeVentura]

yea, none of this code should be in draw()

if (confirmedClicked){
     rendering = createGraphics(250, 250, P3D);  //size of render

    vis = new RenderControler(100,100,100);
    vis.ResetCamera();
    STLParser parser = new STLParser(STLFile);
    ArrayList<Facet> data = parser.parseSTL();
    /*test = new Model(data, .1, .1);
    test.Slice();      // Create gcode in Model object
    gcode = test.getGCode();  

[ZBoylan]

Console window is good

Now need status label

[ZBoylan]

@GeorgeVentura rendering = createGraphics(250, 250, P3D); //size of render // P3D Should we stick that into the confirm file button event handler?

My laptop fan goes full speed and battery drops like crazy when running the UI. I wonder if it has to do with having createGraphics() in draw()?

[GeorgeVentura]

rendering = createGraphics(250, 250, P3D); //size of render // P3D Should we stick that into the confirm file button event handler?

yea

[zbhujwala]

It looks like our status label has attended to the original checklist I posted up here with the exception of Rendering (not sure when Rendering team will be done nor do I know if they throw exceptions/errors to be caught so).

Should we close this issue and focus on documentation/testing now?

[ZBoylan]

Yeah, let's close it.