Skip to content

Conversation

@cerna
Copy link
Contributor

@cerna cerna commented Oct 21, 2020

This pull request merges in changes from upstream LinuxCNC@master branch at commit ca06fc2 (the latest commit with fully passed tests at LinuxCNC's buildbot operation).

I have been talking about rebuilding and upgrading the packages in #6, however the situation ended up so that enough time passed to justify merging in changes beforehand.

Checked locally in simulation mode on amd64.

c-morley and others added 30 commits October 5, 2020 18:38
they needed to be parented fom the main window.
Added objectnames too
For reasons I couldn't quite figure out, the original design had
so many layers that stylesheets changes on the main window
would not change the SoftInputWidget's style.
probably a parenting problem (or based on QObject vrs QWidget).
Simplified now and only two widgets.
Seems to work now.
they needed to be parented fom the main window.
Added objectnames too
Shows the value of an option HAL input pin.
You can set a text template for rich text/formatted display
and add some color to them with rich text
After fixing dialogs to be able to be styled - there didn't llok good.
Now I specifically style them.
It seems on HAL float input pins that got their value
from QPin's QTSignal, would get an integer number rather then the float.
I could change that by changing the order of the overloaded pyqtSignal
definition but then that may have just made eveything a float...

Now we send a PyQt_PyObject that is not supposed to be converted
internally. It seems to work.
1) print command lines settings
   usage example: 5axisgui pivot_len=300
2) parameterize pivot_len incorporating za,zb box heights
3) remove unused tool_radius var
4) pivot_len var now represents correctly the effective pivot_len
   with default value of 250 and is unchanged for compatibility
   with hal pin default of 5axiskins.c

(no functional changes intended by this commit)
Inverting the step pin in the gui actually inverted the direction pin.
Looks like do to python lists not having consistant ordering.
Thank Peter for heads up.
I've needed to test something when i couldn;t see the screen.
Now I can connect a HAL pin to this test LED and have it speak
the (editable) label and state.
1) 5axiskins: use IN pin for pivot_len (not IO)
2) 5axisgui:  provide OUT pin for gui simulated pivot length
3) configs/sim/5axis/bridgemill sim config:
   5axis.tbl: use metric tool length values to agree with 5axis.ini
   5axisgui.hal (new)
     a) compute pivot length needed for 5axiskins with loaded tool
     b) consolidate postgui hal statements
   5axis.ini:
     a) use added 5axisgui.hal file
     b) add omitted [TRAJ]MAX_ANGULAR_VELOCITY
     c) add tooledit options 'z diam'
     d) update j5/w, j2,z limits
   README: demo example
Fix rolfmill example could be tested now without additional moves
Replaced the component type magic numbers with an enumerated type.
The new types have the same values as before.
i.e.
0 == user
1 == realtime
2 == something else (not sure where this value is ever assigned)

There should be no functional change.
…c_numbers

hal: Remove component type magic numbers
The files moved and got renamed back in 2004, this changes the naming
within the file itself.
dngarrett and others added 23 commits October 16, 2020 05:52
Don't error out each time it's selected, but don't set the increments either.
We should add some more checking while parsing the INI actually
…asmaC files and error messages to align with user-doc changes.
plasmac: update user-docs and update/add images, update supporting Pl…
…asmaC files and error messages to align with user-doc changes.
torch on is automatic
Also add an option to read out all register values.


Signed-off-by: andypugh <andy@bodgesoc.org>
changed pocket_number to mod_pocket in direction logic
fix torch on before pierce height error
fix motion start after cut height error
fix offset counts
@zultron
Copy link
Contributor

zultron commented Oct 23, 2020

Thanks for keeping this up to date!

@zultron zultron merged commit 42bf973 into machinekit:machinekit/master Oct 23, 2020
@cerna cerna deleted the merge-21-october-2020 branch October 23, 2020 19:10
@cerna
Copy link
Contributor Author

cerna commented Oct 26, 2020

EMCApplication from this version is uploaded in the Cloudsmith repository.

The armhf version requires a special version of Machinekit-HAL (as @the-snowwhite wanted) which is also in the Machinekit-HAL Cloudsmith repository. (Tested in foreign architecture docker environment.)

Sorry it took so long!

@the-snowwhite
Copy link
Contributor

Thanks
I seem to be stuck in attempting to get anywhere with reviving the mkwrapper functionality before I can get to any real testing...

@the-snowwhite
Copy link
Contributor

tested to fix the setuid issue on armhf
Great to also see the mksocfpga packages

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.