Another plasma component...

More
06 Nov 2018 10:56 #120094 by rodw
Replied by rodw on topic Another plasma component...

phillc54 wrote: It appears that reverse run and the tp fix conflict with each other and it is above my pay grade to fix it (I did try but made a complete hash of it) so it may need robEllenberg to sort it as I think he is the author of both.
In the meantime we need to decide on one of two options until this can be fixed:
1. master with the tp fix + external offsets
2. master before the tp fix + external offsets + reverse run

Cheers, Phill.


My vote would be for the TP fix. The tp bug can affect cut quality, reverse run is more a convenience. Maybe you can make a comment on one of the issues or on the developers mailing list. I think external offsets will be pushed to master soon. Hopefully, they will also look at reverse run. Its been talked about.

The other feature mentioned that may be useful was the state tags branch. This has been suggested to be a possible solution to knowing the original feed rate instead of using remapping. I've looked at the code and have seen the new structure but don't know how you would access it from the component.... Some discussion on Issue 440 here but I could never find out how it might be used.:
github.com/LinuxCNC/linuxcnc/issues/440
The following user(s) said Thank You: phillc54

Please Log in or Create an account to join the conversation.

More
06 Nov 2018 11:15 #120095 by rodw
Replied by rodw on topic Another plasma component...
Phill, I added a note on Issue 447. Hopefully it will get seen even though the issue is closed....
github.com/LinuxCNC/linuxcnc/issues/447
The following user(s) said Thank You: phillc54

Please Log in or Create an account to join the conversation.

More
06 Nov 2018 19:22 #120115 by rene-dev
seen it, I try to merge it.
The following user(s) said Thank You: phillc54, tommylight, rodw

Please Log in or Create an account to join the conversation.

More
06 Nov 2018 22:11 #120124 by phillc54
Rod Wrote:

The other feature mentioned that may be useful was the state tags branch. This has been suggested to be a possible solution to knowing the original feed rate instead of using remapping. I've looked at the code and have seen the new structure but don't know how you would access it from the component.... Some discussion on Issue 440 here but I could never find out how it might


In normal use with this config (with the F word remap) the original feed rate comes from the feed rate set in the gladevcp panel.

Cheers, Phill
The following user(s) said Thank You: rodw

Please Log in or Create an account to join the conversation.

More
07 Nov 2018 04:02 #120142 by rene-dev
I just merged reverse run into master, please test if everything works.
external offsets is next... where can I find the latest version of that?

Rene
The following user(s) said Thank You: phillc54, tommylight

Please Log in or Create an account to join the conversation.

More
07 Nov 2018 04:46 #120149 by phillc54
Thanks Rene, much appreciated.
I will check this out later today.
External offsets is the dgarr/external_offsets branch.

Cheers, Phill
The following user(s) said Thank You: tommylight

Please Log in or Create an account to join the conversation.

More
07 Nov 2018 05:20 #120151 by phillc54
I have just pushed another update...

The F word remapping is now scales correctly for all imperial/metric combinations.
THC calculations only use the user requested feed rate.

Cheers, Phill.
The following user(s) said Thank You: tommylight

Please Log in or Create an account to join the conversation.

More
07 Nov 2018 05:30 #120153 by rene-dev
ok, scratch that. I just had to revert, as there are problems. working on it...
The following user(s) said Thank You: phillc54, tommylight, rodw

Please Log in or Create an account to join the conversation.

More
07 Nov 2018 10:25 #120159 by rodw
Replied by rodw on topic Another plasma component...

rene-dev wrote: I just merged reverse run into master, please test if everything works.
external offsets is next... where can I find the latest version of that?

Rene


Rene, On 20 October, on the developers email list, Dewey said

>The current single patch is:
i updated the commit message so the current single commit is 784db0335:
github.com/LinuxCNC/linuxcnc/commit/784d...5362a0dae498290d8786


Not sure that is useful for you. Thank you very much for committing these things. It will make life easy for many of us!

Please Log in or Create an account to join the conversation.

More
07 Nov 2018 19:25 #120195 by Grotius
Replied by Grotius on topic Another plasma component...
Rod,

SQL is working in Linuxcnc. For making a c component mysql.h is missing, it's not in the include directory, so this takes more time.
We can also use different version's of SQL. lite_version

Maybe the best way is to test a dynamic input set up.

Please Log in or Create an account to join the conversation.

Time to create page: 0.157 seconds
Powered by Kunena Forum