vector-effects: non-scalable-stroke, applied to stroke of bbox

README.md

@@ -1,6 +1,5 @@
 OctoPrint
 =========
-What I did was take the MrBeam fork of Octoprint (because I have a MrBeam, and I like it) - and modify it for pen plotting with an axidraw v3 clone.
 
 [![Flattr this git repo](http://api.flattr.com/button/flattr-badge-large.png)](https://flattr.com/submit/auto?user_id=foosel&url=https://github.com/foosel/OctoPrint&title=OctoPrint&language=&tags=github&category=software)
 

src/octoprint/plugins/softwareupdate/scripts/git-pull.py

@@ -97,7 +97,7 @@ def update_source(git_executable, folder, target, force=False):
 			raise RuntimeError("Could not update, \"git reset --hard\" failed with returncode %d: %s" % (returncode, stdout))
 
 	print(">>> Running: git pull")
-	returncode, stdout = _git(["pull", "origin", "stable-1.2.2"], folder, git_executable=git_executable)
+	returncode, stdout = _git(["pull"], folder, git_executable=git_executable)
 	if returncode != 0:
 		raise RuntimeError("Could not update, \"git pull\" failed with returncode %d: %s" % (returncode, stdout))
 	print(stdout)

src/octoprint/plugins/svgtogcode/__init__.py

@@ -270,7 +270,7 @@ class SvgToGcodePlugin(octoprint.plugin.SlicerPlugin,
 			feedrate = max(1,data["defaultFeedrate"])
 			s.set(["defaultFeedrate"], feedrate)
 		if "svgDPI" in data and data["svgDPI"]:
-			s.set_int(["svgDPI"], data["svgDPI"])
+			s.set(["svgDPI"], data["svgDPI"])
 		if "debug_logging" in data:
 			old_debug_logging = s.get_boolean(["debug_logging"])
 			new_debug_logging = data["debug_logging"] in octoprint.settings.valid_boolean_trues

src/octoprint/plugins/svgtogcode/static/js/convert.js

@@ -41,10 +41,8 @@ $(function(){
 		// image engraving stuff
 		// preset values are a good start for wood engraving
 		self.images_placed = ko.observable(false);
-		self.text_placed = ko.observable(false);
 		self.show_image_parameters = ko.computed(function(){
-			return (self.images_placed() || self.text_placed()
+			return self.images_placed() || (self.fill_areas() && self.show_vector_parameters());
-				|| (self.fill_areas() && self.show_vector_parameters()));
 		});
 		self.imgIntensityWhite = ko.observable(0);
 		self.imgIntensityBlack = ko.observable(500);
@@ -87,7 +85,6 @@ $(function(){
 			self.show_vector_parameters(self.workingArea.getPlacedSvgs().length > 0);
 			self.show_fill_areas_checkbox(self.workingArea.hasFilledVectors())
 			self.images_placed(self.workingArea.getPlacedImages().length > 0);
-			self.text_placed(self.workingArea.hasTextItems());
 			//self.show_image_parameters(self.workingArea.getPlacedImages().length > 0);
 
 			if(self.show_vector_parameters() || self.show_image_parameters()){

src/octoprint/plugins/svgtogcode/static/js/drag_scale_rotate.js

@@ -76,13 +76,13 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 	Snap.plugin( function( Snap, Element, Paper, global ) {
 
 		var ftOption = {
-			handleFill: "red",
+			handleFill: "blue",
 			handleStrokeDashPreset: [5,5],
 			handleStrokeWidth: 2,
 			handleLength: 18,
 			handleRadius: 16,
 			unscale: 1,
-			handleStrokeDash: "5,5",
+			handleStrokeDash: "5,5"
 		};
 
 		Element.prototype.ftToggleHandles = function(){
@@ -100,14 +100,15 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 
 			var rotateDragger = this.paper.select('#userContent').circle(bb.cx + bb.width/2 + ftOption.handleLength * ftOption.unscale, bb.cy, ftOption.handleRadius * ftOption.unscale ).attr({ fill: ftOption.handleFill });
 			var translateDragger = this.paper.select('#userContent').circle(bb.cx, bb.cy, ftOption.handleRadius * ftOption.unscale).attr({ fill: ftOption.handleFill });
+			var resizeDragger = this.paper.select('#userContent').circle(bb.cx + bb.width/2, bb.cy - bb.height/2, ftOption.handleRadius * ftOption.unscale).attr({ fill: ftOption.handleFill});
 
 			var joinLine = freetransEl.ftDrawJoinLine( rotateDragger, ftOption.handleStrokeWidth * ftOption.unscale);
-			var handlesGroup = this.paper.select('#userContent').g( joinLine, rotateDragger, translateDragger );
+			var handlesGroup = this.paper.select('#userContent').g( joinLine, rotateDragger, translateDragger, resizeDragger );
 
 			freetransEl.data( "handlesGroup", handlesGroup );
 			freetransEl.data( "joinLine", joinLine);
 
-			freetransEl.data( "scaleFactor", calcDistance( bb.cx, bb.cy, rotateDragger.attr('cx'), rotateDragger.attr('cy') ) );
+			freetransEl.data( "scaleFactor", calcDistance( bb.cx, bb.cy, resizeDragger.attr('cx'), resizeDragger.attr('cy') ) );
 
 			translateDragger.drag(
 				elementDragMove.bind( translateDragger, freetransEl ),
@@ -118,6 +119,13 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 			freetransEl.unclick();
 			freetransEl.data("click", freetransEl.click( function() {  this.ftRemoveHandles() } ) );
 
+			resizeDragger.drag(
+				dragHandleResizeMove.bind(resizeDragger,freetransEl),
+				dragHandleResizeStart.bind(resizeDragger, freetransEl),
+				dragHandleResizeEnd.bind(resizeDragger, freetransEl)
+
+			);
+
 			rotateDragger.drag(
 				dragHandleRotateMove.bind( rotateDragger, freetransEl ),
 				dragHandleRotateStart.bind( rotateDragger, freetransEl  ),
@@ -207,8 +215,9 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 
 			// transformed bbox
 			this.data("bbT", this.paper.rect( rectObjFromBB( this.getBBox(1) ) )
-							.attr({ fill: "none", stroke: ftOption.handleFill, strokeWidth: ftOption.handleStrokeWidth, strokeDasharray: ftOption.handleStrokeDashPreset.join(',') })
+				.attr({ fill: "none", 'vector-effect': "non-scaling-stroke", stroke: ftOption.handleFill, strokeWidth:  ftOption.handleStrokeWidth, strokeDasharray: ftOption.handleStrokeDashPreset.join(',') })
-							.transform( this.transform().global.toString() ) );
+				.transform( this.transform().global.toString() ) );
+
 			// outer bbox
 			this.data("bb", this.paper.select('#userContent').rect( rectObjFromBB( this.getBBox() ) )
 							.attr({ fill: "none", stroke: 'gray', strokeWidth: ftOption.handleStrokeWidth, strokeDasharray: ftOption.handleStrokeDash }) );
@@ -281,6 +290,34 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 	function elementDragEnd( mainEl, dx, dy, x, y ) {
 	};
 
+	function dragHandleResizeStart( mainElement ) {
+		this.ftStoreStartCenter();
+	};
+
+	function dragHandleResizeEnd( mainElement ) {
+	};
+
+
+	function dragHandleResizeMove( mainEl, dx, dy, x, y, event){
+		var handle = this;
+		var mainBB = mainEl.getBBox();
+		var unscale = mainEl.data('unscale');
+
+		//todo was ist ocx
+		handle.attr({ cx:+handle.data('ocx') + dx*unscale, cy: +handle.data('ocy')+dy*unscale});
+
+		var distance = calcDistance(mainBB.cx,mainBB.cy,handle.attr('cx'),handle.attr('cy'));
+		var scale = distance / mainEl.data("scaleFactor");
+		if(event.shiftKey){
+			scale = Math.round(scale*4) / 4;
+		}
+
+		mainEl.data("scale", scale );
+
+		mainEl.ftUpdateTransform();
+
+	};
+
 	function dragHandleRotateStart( mainElement ) {
 		this.ftStoreStartCenter();
 	};
@@ -297,19 +334,12 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 
 		if(!mainEl.data('block_rotation')){
 			var angle = Snap.angle( mainBB.cx, mainBB.cy, handle.attr('cx'), handle.attr('cy') ) - 180;
-			if(event.shiftKey){
+			if(event.shiftKey){ //enable block rotation
 				angle = Math.round(angle/30) * 30;
 			}
 			mainEl.data("angle", angle );
 		}
 
-		var distance = calcDistance( mainBB.cx, mainBB.cy, handle.attr('cx'), handle.attr('cy') );
-		var scale = distance / mainEl.data("scaleFactor");
-		if(event.shiftKey){
-			scale = Math.round(scale*4) / 4;
-		}
-		mainEl.data("scale", scale );
-
 		mainEl.ftUpdateTransform();
 		mainEl.ftDrawJoinLine( handle );
 	};

src/octoprint/plugins/svgtogcode/static/js/matrix_oven.js

@@ -30,15 +30,14 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 	 */
 	Element.prototype.bake = function (toCubics, dec) {
 		var elem = this;
-
+		if (!elem || !elem.paper) // don't handle unplaced elements. this causes double handling.
-		if (!elem || !elem.paper || elem.type !== "text" || elem.type !== "#text" || elem.type !== "tspan"){
 			return;
-		} // don't handle unplaced elements. this causes double handling.
 
 		if (typeof (toCubics) === 'undefined')
 			toCubics = false;
 		if (typeof (dec) === 'undefined')
 			dec = 5;
+		//var children = elem.selectAll('*')
 		var children = elem.children();
 		if (children.length > 0) {
 			for (var i = 0; i < children.length; i++) {
@@ -55,15 +54,14 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 			elem.type !== "polygon" &&
 			elem.type !== "polyline" &&
 			elem.type !== "image" &&
-			elem.type !== "path" &&
+			elem.type !== "path"){
-			elem.type !== "text" &&
+			
-			elem.type !== "tspan" &&
+//			if(elem.type !== 'g' && elem.type !== 'desc' && elem.type !== 'defs')
-			elem.type !== "#text"){
+//				console.log('skipping unsupported element ', elem.type);
 			return;
 		}
 
-
+		if (elem.type == 'image'){
-		if (elem.type == 'image' || elem.type == "text" || elem.type == "#text"){
 			// TODO ... 
 			var x = parseFloat(elem.attr('x')),
 				y = parseFloat(elem.attr('y')),
@@ -91,6 +89,7 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 			return;
 		}
 
+		//if(elem.type !== 'path') console.log("bake: converting " + elem.type + " to path");
 		var path_elem = elem.convertToPath();
 
 		if (!path_elem || path_elem.attr('d') === '' || path_elem.attr('d') === null)
@@ -439,19 +438,6 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 		// Possibly the cubed root of 6, but 1.81 works best
 		var num = 1.81;
 		var tag = old_element.type;
-
-		var convertMMtoPixel = function (val) {
-			attrList = ['rx','ry','r','cx','cy','x1','x2','y1','y2','x','y','width','height'];
-			for(var attrIdx in attrList) {
-				if(val.attr(attrList[attrIdx]) != null && val.attr(attrList[attrIdx]).indexOf('mm') > -1) {
-					var tmp = parseFloat(val.attr(attrList[attrIdx])) * 3.5433;
-					val.attr(attrList[attrIdx], tmp);
-				}
-			}
-		}
-
-		convertMMtoPixel(old_element);
-
 		switch (tag) {
 			case 'ellipse':
 			case 'circle':

src/octoprint/plugins/svgtogcode/static/js/render_fills.js

@@ -50,7 +50,7 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 				}
 			}
 		} else {
-			if(elem.type === 'image' || elem.type === "text" || elem.type === "#text"){
+			if(elem.type === 'image'){
 				selection.push(elem);
 			} else {
 				if(fillPaths && elem.is_filled()){
@@ -74,10 +74,7 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 			elem.type !== "line" &&
 			elem.type !== "polygon" &&
 			elem.type !== "polyline" &&
-			elem.type !== "path" //&&
+			elem.type !== "path" ){
-			// elem.type !== "text" &&
-			// elem.type !== "#text"
-		){
 			
 			return false;
 		}
@@ -124,7 +121,7 @@ Snap.plugin(function (Snap, Element, Paper, global) {
 
 		// get svg as dataUrl
 		var svgStr = elem.outerSVG();
-		var svgDataUri = 'data:image/svg+xml;base64,' + window.btoa(unescape(encodeURIComponent(svgStr))); //deprecated unescape needed!
+		var svgDataUri = 'data:image/svg+xml;base64,' + window.btoa(svgStr);
 		var source = new Image();
 		source.src = svgDataUri;
 

src/octoprint/plugins/svgtogcode/static/js/working_area.js

@@ -107,7 +107,6 @@ $(function(){
 		self.clear = function(){
 			snap.selectAll('#userContent>*').remove();
 			snap.selectAll('#placedGcodes>*').remove();
-			snap.selectAll('rect:not(#coordGrid)').remove();
 			self.placedDesigns([]);
 		};
 
@@ -133,14 +132,13 @@ $(function(){
 		};
 
 		self.getXYCoord = function(evt){
-			if(/firefox/.test(navigator.userAgent.toLowerCase())) {
+			if(isFirefox) {
 				var scale = evt.target.parentElement.transform.baseVal[0].matrix.a;
 				var x = self.px2mm(evt.offsetX) * scale;
 				var y = self.px2mm(parseFloat(evt.target.attributes.height.value) - evt.offsetY) * scale;
-			} else
+			} else {
-			{
 				var x = self.px2mm(evt.offsetX);
-				var y = self.px2mm(parseFloat(evt.target.farthestViewportElement.clientHeight) - evt.offsetY);
+				var y = self.px2mm(evt.target.ownerSVGElement.getBoundingClientRect().height - evt.offsetY); // hopefully this works across browsers
 			}
 			x = Math.min(x, self.workingAreaWidthMM());
 			y = Math.min(y, self.workingAreaHeightMM());
@@ -243,26 +241,11 @@ $(function(){
 			var url = self._getSVGserveUrl(file);
 			callback = function (f) {
 				var newSvgAttrs = {};
-				if(f.select('svg') == null){
+				var root_attrs = f.select('svg').node.attributes;
-					root_attrs = f.node.attributes;
-				} else {
-					var root_attrs = f.select('svg').node.attributes;
-				}
 				var doc_width = null;
 				var doc_height = null;
 				var doc_viewbox = null;
 
-				// find clippath elements
-				var clipPathEl = f.selectAll('clipPath');
-				if(clipPathEl.length != 0){
-					console.warn("Warning: removed unsupported clipPath element in SVG");
-					self.svg_contains_clipPath_warning();
-					clipPathEl.remove()
-				}
-
-				// find all elements with "display=none" and remove them
-				f.selectAll("[display=none]").remove()
-
 				// iterate svg tag attributes
 				for(var i = 0; i < root_attrs.length; i++){
 					var attr = root_attrs[i];
@@ -278,19 +261,9 @@ $(function(){
 					}
 				}
 
-				// find Illustrator comment and notify
-				f.node.childNodes.forEach(function(entry) {
-					if(entry.nodeType == 8) { // Nodetype 8 = comment
-						if(entry.textContent.indexOf('Illustrator') > -1) {
-							new PNotify({title: gettext("Illustrator SVG Detected"), text: "Illustrator SVG detected! To preserve coorect scale, please go to the \'Settings\' menu and change the \'SVG dpi\' field under \'Plugins/Svg Conversion\' according to your file. And add it again.", type: "info", hide: false});
-						}
-					}
-				});
-
 				// scale matrix
 				var mat = self.getDocumentViewBoxMatrix(doc_width, doc_height, doc_viewbox);
-				var dpiscale = 90 / self.settings.settings.plugins.svgtogcode.svgDPI();
+				var scaleMatrixStr = new Snap.Matrix(mat[0][0],mat[0][1],mat[1][0],mat[1][1],mat[0][2],mat[1][2]).toTransformString();
-				var scaleMatrixStr = new Snap.Matrix(mat[0][0],mat[0][1],mat[1][0],mat[1][1],mat[0][2],mat[1][2]).scale(dpiscale).toTransformString();
 				newSvgAttrs['transform'] = scaleMatrixStr;
 
 				var newSvg = snap.group(f.selectAll("svg>*"));
@@ -416,20 +389,8 @@ $(function(){
 			};
 		};
 
-
-		self.svg_contains_clipPath_warning = function(){
-            var error = "<p>" + gettext("The SVG file contains clipPath elements.<br/>clipPath is not supported yet and has been removed from file.") + "</p>";
-            //error += pnotifyAdditionalInfo("<pre>" + data.jqXHR.responseText + "</pre>");
-            new PNotify({
-                title: "clipPath elements removed",
-                text: error,
-                type: "warn",
-                hide: false
-            });
-		};
-
 		self.svg_contains_text_warning = function(svg){
-            var error = "<p>" + gettext("The SVG file contains text elements.<br/>If you want to laser just their outlines,<br/>please convert them to paths.<br/>Otherwise they will be engraved with infill.") + "</p>";
+            var error = "<p>" + gettext("The svg file contains text elements.<br/>Please convert them to paths.<br/>Otherwise they will be ignored.") + "</p>";
             //error += pnotifyAdditionalInfo("<pre>" + data.jqXHR.responseText + "</pre>");
             new PNotify({
                 title: "Text elements found",
@@ -437,6 +398,7 @@ $(function(){
                 type: "warn",
                 hide: false
             });
+			svg.selectAll('text,tspan').remove();
 		};
 
 		self.svg_misfitting_warning = function(svg, misfitting){
@@ -742,16 +704,6 @@ $(function(){
 			return snap.selectAll("#userContent image");
 		};
 
-		self.hasTextItems = function () {
-			if(snap.selectAll("#userContent tspan").length > 0 ||
-				snap.selectAll("#userContent text").length > 0 ||
-				snap.selectAll("userContent #text").length > 0) {
-				return true
-			}else{
-				return false
-			}
-		};
-
 		self.getPlacedGcodes = ko.computed(function() {
 			var gcodeFiles = [];
 			ko.utils.arrayForEach(self.placedDesigns(), function(design) {
@@ -874,7 +826,7 @@ $(function(){
 				for (var i = 0; i < fillings.length; i++) {
 					var item = fillings[i];
 
-					if (item.type === 'image' || item.type === "text" || item.type === "#text") {
+					if (item.type === 'image') {
 						// remove filter effects on images for proper rendering
 						var style = item.attr('style');
 						if (style !== null) {

src/octoprint/printer/standard.py

@@ -331,7 +331,7 @@ class Printer(PrinterInterface, comm.MachineComPrintCallback):
 	def position(self, x, y):
 		printer_profile = self._printerProfileManager.get_current_or_default()
 		movement_speed = min(printer_profile["axes"]["x"]["speed"], printer_profile["axes"]["y"]["speed"])
-		self.commands(["G90", "G0 X%.3f Y%.3f F%d" % (x, y, movement_speed)])
+		self.commands(["G90", "G0 X%.3f Y%.3f F%d" % (x, y, movement_speed), "?"])
 
 	def _convert_rate_value(self, factor, min=0, max=200):
 		if not isinstance(factor, (int, float, long)):

src/octoprint/templates/mrbeam_index.jinja2

@@ -1,7 +1,7 @@
 <!DOCTYPE html>
 <html>
     <head>
-        <title data-bind="text: title">Mr Draw</title>
+        <title data-bind="text: title">Mr Beam</title>
 
         <link rel="shortcut icon" href="{{ url_for('static', filename='img/favicon.png') }}">
         <link rel="apple-touch-icon" sizes="114x114" href="{{ url_for('static', filename='img/apple-touch-icon-114x114.png') }}">
@@ -16,7 +16,7 @@
 			<div class="container">
 				<div class="navbar-header brand" style="min-width: 272px;">
 					<a class="navbar-brand" href="#">
-						<img alt="Mr Draw Logo" src="{{ url_for('static', filename='img/mr-draw-red_x120.png') }}">
+						<img alt="Mr Beam Logo" src="{{ url_for('static', filename='img/mr-typo-red_x120.png') }}">
 					</a>
 				</div>
 				<!-- Navbar -->
@@ -58,7 +58,7 @@
 
 									<div id="control" class="accordion-inner" data-bind="visible: isReady() || isLocked() || isFlashing()">
 										<div data-bind="visible: isLocked ">
-											Mr Draw is in a locked state as it does not know its position.
+											Mr Beam is in a locked state as it does not know its position.
 											First remove any objects blocking the gantry's travel range.
 											Then do a homing cycle.
 											<div style='text-align: center; padding:.5em;'>
@@ -657,10 +657,12 @@
 							<div class="span4">
 								<ul class="focus_steps">
 									<li>1. Place your material on the working area</li>
-									<li>2. Move the pen over the material</li>
+									<li>2. Move the laser over the material</li>
-									<li>&RightArrow; Now enable the pen calibration mode</li>
+									<li>3. Put on your safety glasses</li>
-									<li>5. Adjust the pen until it touches the paper</li>
+									<li>4. Turn the laser safety switch to on</li>
-									<li>&RightArrow; Disable the pen calibration mode. </li>
+									<li>&RightArrow; Now enable the focus mode</li>
+									<li>5. Adjust the focus until the laser beam is as small as possible</li>
+									<li>&RightArrow; Disable the focus mode. </li>
 								</ul>
 								<div style="text-align:center">
 								<div class="btn-group" role="group" aria-label="focus mode control" style="">

src/octoprint/util/comm_acc2.py

@@ -88,7 +88,7 @@ class MachineCom(object):
 		self._finished_passes = 0
 
 		# regular expressions
-		self._regex_command = re.compile("^\s*\$?([GM]\d+|[THX])")
+		self._regex_command = re.compile("^\s*\$?([GM]\d+|[TH])")
 		self._regex_feedrate = re.compile("F\d+", re.IGNORECASE)
 		self._regex_intensity = re.compile("S\d+", re.IGNORECASE)
 
@@ -147,7 +147,7 @@ class MachineCom(object):
 				elif line.startswith('['): # feedback message
 					self._handle_feedback_message(line)
 				elif line.startswith('Grb'): # Grbl startup message
-					self._handle_startup_message(line)
+					self._handle_startup_message()
 			except:
 				self._logger.exception("Something crashed inside the monitoring loop, please report this to Mr. Beam")
 				errorMsg = "See octoprint.log for details"
@@ -405,7 +405,7 @@ class MachineCom(object):
 		elif line[1:].startswith('Dis'): # [Disabled]
 			pass
 
-	def _handle_startup_message(self, line):
+	def _handle_startup_message(self):
 		if self.isOperational():
 			errorMsg = "Machine reset."
 			self._cmd = None
@@ -420,14 +420,7 @@ class MachineCom(object):
 			self._changeState(self.STATE_LOCKED)
 			eventManager().fire(Events.ERROR, {"error": self.getErrorString()})
 		else:
-			versionMatch = re.search("Grbl (?P<grbl>.+?)(_(?P<git>[0-9a-f]{7})(?P<dirty>-dirty)?)? \[.+\]", line)
+			self._onConnected(self.STATE_LOCKED)
-			if versionMatch:
-				versionDict = versionMatch.groupdict()
-				self._writeGrblVersionToFile(versionDict)
-				if self._compareGrblVersion(versionDict) is False:
-					self._flashGrbl()
-				else:
-					self._onConnected(self.STATE_LOCKED)
 
 	def _update_grbl_pos(self, line):
 		# line example:
@@ -618,7 +611,7 @@ class MachineCom(object):
 		params = ["avrdude", "-patmega328p", "-carduino", "-b" + str(self._baudrate), "-P" + str(self._port), "-D", "-Uflash:w:" + pathToGrblHex]
 		rc = subprocesscall(params)
 
-		if rc == 0:
+		if rc is False:
 			self._log("successfully flashed new grbl version")
 			self._openSerial()
 			self._changeState(self.STATE_CONNECTING)
@@ -755,10 +748,6 @@ class MachineCom(object):
 		self._changeState(self.STATE_HOMING)
 		return cmd
 
-	def _gcode_X_sent(self, cmd, cmd_type=None):
-		self._changeState(self.STATE_HOMING) # TODO: maybe change to seperate $X mode
-		return cmd
-
 	def _gcode_Hold_sent(self, cmd, cmd_type=None):
 		self._changeState(self.STATE_PAUSED)
 		return cmd
@@ -1169,9 +1158,6 @@ class PrintingGcodeFileInformation(PrintingFileInformation):
 
 		if not os.path.exists(self._filename) or not os.path.isfile(self._filename):
 			raise IOError("File %s does not exist" % self._filename)
-
-		self._stripCommments()
-
 		self._size = os.stat(self._filename).st_size
 		self._pos = 0
 
@@ -1225,16 +1211,6 @@ class PrintingGcodeFileInformation(PrintingFileInformation):
 			self._logger.exception("Exception while processing line")
 			raise e
 
-	def _stripCommments(self):
-		dir     = os.path.dirname(os.path.abspath(self._filename))
-		tmpfile = open(dir + '/gcode.tmp', 'w')
-		with open(self._filename, "r") as fileobject:
-			for line in fileobject:
-				if process_gcode_line(line) is not None:
-					tmpfile.write(line)
-		tmpfile.close()
-		self._filename = dir + '/gcode.tmp'
-
 def convert_pause_triggers(configured_triggers):
 	triggers = {
 		"enable": [],
@@ -1299,7 +1275,6 @@ def serialList():
 	baselist = baselist \
 				+ glob.glob("/dev/ttyUSB*") \
 				+ glob.glob("/dev/ttyACM*") \
-				+ glob.glob("/dev/ttyAMA*") \
 				+ glob.glob("/dev/tty.usb*") \
 				+ glob.glob("/dev/cu.*") \
 				+ glob.glob("/dev/cuaU*") \