Remove Report for testing

Use Arc_Milestone for arc calculations - improve accuracy of rt_axis0 and rt_axis1

Also added more tests for comparison in the temporary report

Put diagnostic info back in

Merge remote-tracking branch 'Smoothieware/edge' into Arc_Diagnosetic2

Merge pull request #1295 from Smoothieware/add/arc_milestone

added arc_milestone

added arc_milestone

Merge pull request #1294 from wolfmanjm/upstreamedge

revert PR #1293

Revert "Merge pull request #1293 from Zaaphod/Missing_Arcs"

This reverts commit 2ed924ed1e5bc6ae2f341f8c22ec3bf3c85c6476, reversing
changes made to 573ed7c0c719008075acb1d36850e718a2707372.

Revert "Update Robot.cpp"

This reverts commit 402a31afb0efab312e3fc2e51d6b3954246c9631.

Update Robot.cpp

replace and with &&

Merge pull request #1293 from Zaaphod/Missing_Arcs

Improvements to G2/G3 Arcs

Remove diagnostic output to console and add comments

Add test tor full circles and only perform atan2 if not a full circle.  fix correction for atan2 to prevent accidental giant circles

Modify dignostic output and perform both old and new atan corrections to see the difference.

fix offset

actually improve the probe result panel screen

Diagnostic variables sent to console for evaluation

clean up cnc panel probe output

Add max_speed parameter and M203 Sxxx to set maximum feedrate

call on_idle

fixes issue #1286

Update config

Merge pull request #1282 from wolfmanjm/upstreamedge

Allow G38 to do arbitrary move to xyz

Allow G38 to do arbitrary move to xyz

Merge pull request #1281 from wolfmanjm/upstreamedge

fix percent complete for panels

FIx percent progress for panels

fix up some style issues in the laser PR

Merge pull request #1278 from wolfmanjm/upstreamedge

fix bad formatting from laser PR

fix bad formattingfrom PR

Merge pull request #1277 from nuclearcat/edge

Add more safety to laser testing by optional duration parameter and s…

Fixing compile warnings, sorry

Properly handling when PWM frequency is slower than 1khz

Add more safety to laser testing by optional duration parameter and status query

update latest bins

Merge pull request #1276 from wolfmanjm/upstreamedge

return Error on exceeding soft limit

return Error on exceeding soft limit

Merge pull request #1149 from Smoothieware/feature/soft-endstops

Feature/soft endstops

Merge pull request #1274 from ahmetipkin/edge

Compansation dampening functionality is added

Fixed indentation and braces

Deleted the bogus files.

Made the Compansation Dampening as optional

If the config values are not set; the compansation dampening is shut down; so the compansation is applied everywhere - like the old behaviour

If you want to enable it; you should define both of

leveling-strategy.rectangular-grid.height_limit
leveling-strategy.rectangular-grid.dampening_start

configs.

Merge pull request #1275 from Zaaphod/Probe-G20

Display results of probe in inches if in G20 mode

Display results of probe in inches if in G20 mode

Added compensation degragation function for rectangular grid

Now the compensation is not applied always - rather is applied only for limited amount (like Repetier firmware does). In order to configure it, you add two new configs:
leveling-strategy.rectangular-grid.height_limit <height_limit>
leveling-strategy.rectangular-grid.dampening_start <start limit>

Basically what it does is; until Z axis is below 'start limit', any compensation that would be applied is applied as is (full power). However, after this 'start limit' is hit, until 'height limit' is reached, the compensation is dampened gradually (linearly) from full power to none (i.e. no compensation after 'height limit' is passed)

Main reason for this is: we generally use Mesh bed leveling for compensating our non-flat beds; but generally after first few layers; the extruded plastic can cover for this bent; and it is no longer needed.

fix merge

Merge remote-tracking branch 'upstream/edge' into feature/soft-endstops

Merge remote-tracking branch 'upstream/edge' into feature/soft-endstops

update latest bins

Merge pull request #1272 from wolfmanjm/upstreamedge

fix arc generation for tiny arcs

fix arc generation for tiny arcs

Merge pull request #1270 from wolfmanjm/upstreamedge

fix usbdevice.

fix typos in USBdevice

make query new format the default

Merge pull request #1269 from cilynx/feature/m115_get_firmware_version

Add M115 Get Firmware Version

Add M115 Get Firmware Version

I based this implementation on the existing `version` command in
SimpleShell as well as looking what the other players are doing with
M115:

http://reprap.org/wiki/G-code#M115:_Get_Firmware_Version_and_Capabilities

http://marlinfw.org/docs/gcode/M115.html

https://duet3d.com/wiki/G-code#M115:_Get_Firmware_Version_and_Capabilities

I did not bring MCU over from `version` as I felt there would be too
much code duplication / complexity for too little gain.  The M115 output
from my test branch looks like this:

FIRMWARE_NAME:Smoothieware, FIRMWARE_URL:http://smoothieware.org,
SOURCE_CODE_URL:https://github.com/Smoothieware/Smoothieware,
FIRMWARE_VERSION:feature/m115_get_firmware_version-8d5ff26,
BUILD_DATE:Feb 16 2018 17:11:15, SYSTEM_CLOCK:100MHz, AXES:3, CNC:1,
MSD:1
ok

Of course FIRMWARE_VERSION will look better when it's `edge` as opposed
to my overly long branch name.

I arbitrarily followed Marlin's example of leaving colons in the value
of the k-v pair as a problem for the parser instead of URL encoding them
like RepRap or avoiding them entirely like Duet3d.  I'm not married to
this, so if there's a preference for another direction, I'm open.

If accepted, I believe this fixes #279.

Merge pull request #1267 from cilynx/fix/spindle_pid_tuning

Fix M958 Set Spindle PID Parameters

Fix M958 Set Spindle PID Parameters

Original code sets the P term to any values provided for P, I, or D with
no way to set I or D terms.  This change sets I and D correctly as one
would expect when I and D are passed to M958.  The set_i_term() and
set_d_term() functions already existed, so the layout in
on_gcode_received() was likely a simple oversight by the original
author.

I tested this change on the bench with a Smoothieboard driving an MC2100
DC controller hooked up to a 2HP treadmill motor.  Prior to the change,
M958 would only set the P term.  After the change, M958 works as
expected and the PID controller can be tuned live.

Merge pull request #1266 from wolfmanjm/upstreamedge

some convenience features

add example six-axis config snippet

add max_travel warning to homing cycle fail

add a useful message after delta grid probe to save it.

Merge pull request #1262 from wolfmanjm/upstreamedge

Fix snprintf return value misunderstanding

sizeof(buf)-1 not needed

fix misunderstanding for return value of snprintf, where it matters.

Update TemperatureControl.cpp

Merge pull request #1256 from wolfmanjm/upstreamedge

fix progress when large files are being played

fix progress when large files are being played

update -latest bins

Merge pull request #1250 from wolfmanjm/fix/disk_write

fix disk write when sector count > 1.

remove 400 byte limit write HACK

Merge pull request #1241 from michaelnew/cartesian_grid_extrapolation

Add simple extrapolation for cartesian grid compensation

fix disk write when sector count > 1.

This should allow the HACK that limits disk writes to 512 bytes.

Merge pull request #1239 from adrianjobst/fix/probing-dwell

fix / dwell before probing

update firmware.bin

Merge pull request #1244 from wolfmanjm/upstreamedge

Fix handling of an illegal line number

Fix crash if an illegal line with just N1 on it is sent or a blank line with line number and checksum
Fixes issue #1243

Merge pull request #1242 from lordofhyphens/stream-log

Alternate quiet mode for smoothie-stream that outputs to a log instead.

Alternate quiet mode that outputs to a log instead (useful when print progress wants to be monitored and process is to be backgrounded).

cartesian grid still provides an offset for out-of-bounds xy locations

moved dwell time to the begining of the function, so the if clause will not cancell the function before the dwell ever happened

Merge remote-tracking branch 'upstream/edge' into upstreamedge

Merge pull request #1237 from cilynx/fix/spindle_analog_report_speed

Report actual RPM and PWM instead of just target

Report actual RPM and PWM instead of just target

The way `report_speed` was implemented, `M957` shows `target_rpm` as the
"Current RPM" and then calculates the PWM "Analog value" from there.

Because `turn_off` uses `update_pwm(0)` to turn off the output signal
without clobbering the `target_rpm` set by `M3`, if you `turn_off` the
spindle (via `M5`, a `HALT` state, etc), `M957` (`report_speed`)
incorrectly continues to report the old "Current RPM" and calculated
"Analog value" even thought the PWM signal has actually been cut off.

I turned this around so `report_speed` reads the real PWM ("Analog
value") being sent out `pwm_pin` and then calculates the real "Current
RPM" from there.  I also appended a new "Target RPM" field to the
`report_speed` output containing `target_rpm`.

Tested over USB serial with `M597`, `M3`, `M5`, `M112`, `M999`, `^X`,
and `$X`.

Merge pull request #1236 from cilynx/fix/spindle_off_on_halt

Fix/spindle off on halt

Add spindle.ignore_on_halt to config sample snippet

Consolidate common spindle functions

Doing the exact same event registration in every sub-module isn't very
DRY and could lead to inadvertent missed registrations in new
sub-modules.  Consolidating the registrations into `SpindleMaker`.

Stop spindle on HALT

When a machine enters a `HALT` state for any reason, the spindle should
stop for safety along with everything else.  This commit changes the
default behavior to `turn_off` the spindle `on_halt`.  I added an
optional `spindle.ignore_on_halt` config parameter that defaults to
`false` in case anyone needs the previous less-safe behavior.

I believe this addresses #1069.

I confirmed functionality on `PWMSpindleControl` via `M957` as well as a
physical scope while causing `HALT` via `M112` or `^X` over the serial
interface.

I found some weirdness in `AnalogSpindleControl` around `report_speed`,
but I'll address that in another PR.  I did confirm that `turn_off` is
being called `on_halt` as it should be and the PWM duty cycle is reduced
to 0 even though `M957` (`report_speed`) continues to report that the
spindle is moving.

Without having the equipment for the other side of the RS485 connection,
I can't do much to test `ModbusSpindleContol`, but
`HuanyangSpindleControl`, the only supported Modbus VFD right now,
implements `turn_off`, so I don't see any reason it shouldn't work fine
with this change.

update cnc firmware latest

move spindle.config

Merge pull request #1235 from cilynx/feature/spindle_max_pwm

Feature/spindle max pwm

Feature/spindle max pwm

Treadmill motor controllers are becoming increasingly popular in DIY
machines.  One such popular controller is the mc2100 which expects a
duty cycle of 0-85% for 0-100% speed.  To keep the PWM signal below the
controller maximum and thus prevent the controller from going into an
error state when higher speeds than physically possible are requested,
we need a config option for the ceiling on the PWM signal.

I tested this for backwards compatibility with my original configuration
without this new parameter included and it works as expected, using 100%
as the maximum duty cycle.

I tested setting `spindle.max_pwm` to 0.85 (above the hard-coded 0.5
starting point) and 0.25 (below the hard-coded 0.5 starting point) and
confirmed expected function for both via M957 output and viewing the
physical PWM output with a scope.

I added a sample `spindle.config` showing this new parameter in context
with the other spindle parameters I could find with a quick grep of the
code and search through the docs.

Merge pull request #1234 from cilynx/fix/get_pid_settings

Align get_pid_settings() with report_settings()

Merge pull request #1233 from cilynx/fix/spindle_on

Remove conflicting bool spindle_on

Align get_pid_settings() with report_settings()

In the handler for M958 in `SpindleControl`, a function,
`get_pid_settings()`, is called which does not appear to exist beyond
its virtual initialization also in `SpindleControl`.  Because this
function does not really exist, M958 does not return any PID data.

`PWMSpindleControl` contains a function `report_settings()`, which
prints the PID data one would expect from M958.  Replacing
`SpindleControl`'s references to the non-existent `get_pid_settings()`
with `report_setting()` causes M958 to return PID data as expected.

Remove conflicting bool spindle_on

Bool `spindle_on` is initialized in `SpindleControl` and is shared
between `SpindleControl` and whichever control method (PWM, Modbus,
Huanyang, or Analog) is configured.

Initializing `spindle_on` in `PWMSpindleControl` as well clobbers the
shared variable and leads to `SpindleControl` and `PWMSpingleControl`
having different values for `spindle_on` and thus the spindle is never
actually turned on when sending M3 commands.

This can be confirmed via M957 which will always show that the current
PWM is 0, no matter what the current and target RPM.

Removing this superfluous initializion causes `spindle_on` from
`SpindleControl` to be shared as expeced and the module then works as
can be confirmed with M957 showing appropriate PWM values responding to
current and target RPM.

Merge pull request #1230 from oddstr13/edge

Added COPYING file

Added COPYING file

update firmware.bin

update latest bins (same as normal bin)

update firmware.bin

Merge pull request #1226 from Zaaphod/Fix_XZ_Arcs

Fix Math for Arcs in XZ Plane - Issue  #1181

Fix Math for Arcs in XZ Plane - Issue  #1181

The math for calculating angular travel needs to be reversed only for XZ arcs, this is because the arc is now starting and ending in different quadrants than a similar arc in the other 2 planes.

Merge pull request #1223 from DavidJRobertson/make-debug-store-win

Fix `make debug-store` (and hence `make upload`) on Windows