`fanuc/{machine}/stat_info/{path}`: Potentially missing properties

[tukusejssirs]

According to the docs of cnc_statinfo, there are four groups based on the Fanuc series:

• all Fanucs should provide this data (base for other groups and all of these are already published):

  • run (Status of automatic operation);
  • edit (Status of program editing);
  • motion (Status of axis movement,dwell);
  • mstb (Status of M, S, T, B function);
  • emergency (Status of emergency);
  • alarm (Status of alarm);

• Series 15/15i (I don’t have access to such machines):

  • aut (AUTOMATIC mode selection);
  • manual (MANUAL mode selection);
  • write (Status of writing backupped memory);
  • labelskip (Status of label skip);
  • warning (Status of warning);
  • battery (Status of battery).

• Series 30i/31i/32i, 0i-D/F:

  • hdck (Status of manual handle re-trace);
  • tmmode (T/M mode selection);
  • aut (AUTOMATIC/MANUAL mode selection).

• Series 16/18/21, 16i/18i/21i, 0i, Power Mate i, PMi-A:

  • tmmode (T/M mode selection);
  • aut (AUTOMATIC/MANUAL mode selection).

• Series 16i/18i-W:

  • aut (AUTOMATIC/MANUAL mode selection).

I suggest to support them all.

Note that some have two separate bytes (or bits?) for automatic and manual mode, others a single one.

Moreover, I suggest to modify the data structure a bit:

"data": {
  "alarm": "(No alarm)"
  "battery": "???",
  "edit": "****(Not editing)",
  "emergency": "(Not emergency)",
  "hdck": "???",
  "labelskip": "???",
  "mode": "automatic"  // automatic | manual
  "mode_type": "MDI"   // Change the property name if you find a better name
  "motion": "***",
  "mstb": "***",
  "run": "STOP",
  "tmmode": "???"
  "warning": "???",
  "write": "???",
}

Just a note: I have access to machines with the following Fanuc versions. I can test this on them.

Fanuc series	Fanuc model	Paths
Fanuc Series 0i-TD	MODEL D	1
Fanuc Series 0i-TD	MODEL D	2
Fanuc Series 0i-TF	MODEL F	1
Fanuc Series 0i-TF Plus	MODEL F	1
Fanuc Series 31i	MODEL B	1

[MRIIOT]

These are the two structures available.

https://github.com/Ladder99/fanuc-driver/blob/12301dad1ca31ccb6e3dafcfcfbdf13bfede95c2/fanuc/fwlib.cs#L3319-L3354

[tukusejssirs]

It is already an improvement, but we still miss some other controller model data. :wink:

I suggest to add into the structure comment (e.g. /* cnc_statinfo:read CNC status information */) also the controller models which support that particular if branch. FS15D might be enough for you, but I still suggest to comment the code properly.

A nitpick: you still don’t use same code style: on L3321 use write Pack=4 (and L3324 SizeConst=2), in L3341 Pack = 4.

[MRIIOT]

fwlib.cs does need to be refactored.