Registrationmarks #25
Conversation
+ Graphtec.py: de-cruft and refactor hardware communication and add proper command delimiter
Add some comments
Merge commit '47d86e1ca3672eef203541da29944d96203b83e6' into registrationmarks
Corrected some typos
… to svg document origin.
Limit cutting area inside regmarks.
* Position of the regmarks correctedd. (They were slightly off, leading to parallelograms) * Width of the regmarks set to 0.5mm, this is hardcoded in sourcecode * Named the regmarks with IDs * Created an extra layer for the regmarks.
|
Hey @EtherGraf thanks for working on regmarks support! I am afraid I broke your PR by allowing other changes to happen meanwhile. |
|
I am no git expert, so I cannot overview the implications of a rebase. Note that my work is based partly on changes from Alexander Senger, I will have a look later. Am 30.08.2016 um 18:54 schrieb Jürgen Weigert:
|
|
Same with PR#21. unfortunately. |
|
I have tried merging yout current master into my local regmarks branch.
It was only 8 lines to edit by hand. If it does help you, I can push
this updated branch to github in the next days.
I am not sure which is the cleanest way to include this branch into your
master as my local repository has lots of trial branches and remerges.
|
|
@EtherGraf I am inviting you to directly push here. How about that? |
|
Just released the merged regmarks branch. I had no time to do any tests, but expect no serious problems. At weekend I will have more time to continue with this. |
|
Hey great work. Thanks. |
|
I remember that something on the return to home was changed with the At the moment I'm a little short at time, so please excuse me till |
# Conflicts: # sendto_silhouette.inx # silhouette/Graphtec.py
Cleanup of code and mirror the coordinate system. This is a prerequisite to get the regmarks working. With this commit the options return_home and no_trailer has been removed. It will be replaced in a later commt with end_position. Comments from the author: The biggest problem was the non-standard communication between software and cutter. There seem to be a lot of left-overs from previous work and also quite a bit of mis-interpretation. My approach: * replace comma with "\x03" as command delimiter (comma is for separation of parameters within command) and properly terminate all communication * drop all unnecessary communication (polls where result is not used, unknown commands, anything not found in the sniffed communication * refactor communication in initialization, setup and job * do more testing to determine actual effect of commands (well, not always successful, but I think it is an improvement) * use cleaner coordinate transform before sending data to Cameo. Helps in offset-issue (see comment in code) This results in much more stable and reliable operation with my cutter. I think other users could also benefit.
Comment from the author: The Cameo will get lost in communication if any new data is send while the machine is busy cutting. Also data chunk size in Silhouette Studio is limited to 3k chars at most and 1k is standard. In response a method "safe_write" was added as a wrapper for the write method to further split large data words (from plot commands) into more digestible chunks. Also inkscape-silhouette will now wait for the cutter to be ready before sending new data. This results in the cutter returning to the main menu after the complete job is finished (e.g. the nifty option to advance the media into a position where the very handy cross-cutter can be used is offered). Without waiting the machine will only offer to eject the media after cutting.
Comment from the author: In GUI and plot code an option is implemented to chose the end position of the blade after cutting. This can be useful for some jobs. There was already some code which seemed to be intended for this purpose but was left unfinished. Frankly, I had some difficulties understanding the logic and implemented my own approach.
Solves pull request #25 Tested and verified working on my Silhouette Portrait.
I have created a branch registrationmarks, in which I got the regmark functionality to work.
My silhouette portrait has succesfully searched and found the registration marks. It's tested with Ubuntu 14.04. No idea if this works on a cameo too. Needs testing.
To get it working, I had to pull in some changes from Alexander Senger, which sets the default coordinate system from landscape to portrait mode.
To plot with registration marks do the following:
The plotter will search the registration marks at the given positions. If it founds the marks, they will serve as accurate reference and define the origin. Therefore it is necessary to set the correct offset values of the mark. As a result the cut will go precisely along the graphics.
A further improvement (not yet implemented) would be automatic detection of the regmarks in the svg by ID, so it would not be necessary to set the numbers by hand.