#linuxcnc-devel Logs
Jun 05 2017
#linuxcnc-devel Calendar
06:16 AM jthornton- is now known as jthornton\
06:33 AM jthornton\ is now known as jthornton
03:40 PM andypugh: Does anyone recall if a problem with a REMAP= that lacks “modalgroup” has been fixed?
03:40 PM andypugh: The system insists that it is happy to use a default value, but then it seems to not do so.
03:41 PM andypugh: (doing a pull-and-make to check)
03:44 PM cradek: I sure don't know anything about that
03:49 PM andypugh: Looks like the bug is still there.
03:49 PM andypugh: Hmm
03:49 PM andypugh: Pity nobody understands the remap code.
03:54 PM jepler: if (r.modal_group == -1) {
03:54 PM jepler: Error("warning: code '%s' : no modalgroup=<int> given, using default group %d : %d:REMAP = %s",
03:54 PM jepler: code, GCODE_DEFAULT_MODAL_GROUP, lineno, inistring);
03:54 PM jepler: r.modal_group = GCODE_DEFAULT_MODAL_GROUP;
03:54 PM jepler: break;
03:55 PM jepler: src/emc/rs274ngc/rs274ngc.hh:93:#define Error(fmt, args...) _logDebug(EMC_DEBUG_UNCONDITIONAL,0,-1,fmt, ## args)
03:55 PM jepler: the source code looks like it does what the comment implies
03:55 PM jepler: // if no modalgroup= was given in the REMAP= line, use these defaults
03:55 PM jepler: #define MCODE_DEFAULT_MODAL_GROUP 10
03:55 PM jepler: #define GCODE_DEFAULT_MODAL_GROUP 1
03:56 PM jepler: so maybe put a how-to-reproduce in a github issue so at least it's on the record
03:59 PM jepler: .. such as, in tests/remap/sequencing/test.ini, change
03:59 PM jepler: -REMAP=G88.1 modalgroup=1 ngc=rg881
03:59 PM jepler: +REMAP=G88.1 ngc=rg881
04:12 PM jepler: andypugh: or take a look at that PR and see if it's about the right thing
04:17 PM andypugh: sorry, i was “hardwareing”. be back later
04:17 PM jepler: np
08:29 PM jepler: we are warned by our hosting provider that there may be loss of network connectivity on flo at 2017-06-13 04:00
09:10 PM seb_kuzminsky: did you tell them we make plenty of those all on our own?
09:46 PM Tom_L: do you warn ahead though?
10:21 PM -!- #linuxcnc-devel mode set to +v by ChanServ