Gmoccapy 3.5.1 #3494

hansu · 2025-07-03T06:45:59Z

Main changes:

Add button "refresh dir" in open file page
Main window / jog:
- Add icon for continuous jog, change alignment of jog increment button box
- Add setting for jog button size + right pane width
- Style velocity/spindle info box (remove S-prefix, add units)
G-code editor:
- Add button to show preview beside editor (split view). The split-state is currently remembered during the session. Not on exit.
- Add comment/uncomment button for G-code + keybinding
- Re-arrange search and replace bar
MDI-entry: bigger enter button (touch-friendly)

And the changes from other PRs which are already in master:

Add button to call calculator in mdi and gcode edit modes (Gmoccapy: Add button to call calculator in mdi and gcode edit modes #3496, thanks Sigma1912)
Fix several warnings. Thanks Sigma1912
Improvements to the calculator widget (Some improvements to the Gladevcp/calculator widget #3473, thanks Sigma1912)
Use calculator widget to edit numeric values in tooltable (Gmoccapy: Use calculator widget to edit numeric values in tooltable #3472, thanks Sigma1912)
Use calculator widget to edit offset values in offsetpage (Gmoccapy: improvements to the offsetpage widget #3457, thanks Sigma1912)
Improvements to the offsetpage widget. Especially add column for rotation in offsets G54..G59.3 (Gmoccapy: improvements to the offsetpage widget #3457, thanks Sigma1912)
Fix issues Gmoccapy - Odd behaviour in G96 (CSS) mode #3447 and Spindle using other speed than in GCode, 2.8 #982 (2.9 Gmoccapy: fix issues #3447 and #982 #3454, thanks Sigma1912)
- Fix the display of very large numbers for spindle speed and 'Vc' in the gui when using G96
- Fix 'spindle.0.speed-out' pins being reset to wrong values when using G96
- Fix wrong spindle speed set when using M1 with optional stop enabled
- The 'Spindle [rpm]' value will now track hal pin 'spindle.0.speed-out'
- The 'S' value in the 'G-Code' frame now tracks 'self.stat.settings[2]'
Fix external_load function -wrong function name - makes external loading of files possible again (Thanks CMorley)

@PeterMue Can you please add resp. recreate the new icons to your https://github.com/PeterMue/gmoccapy-icon-themes? For those I get weird colors, but I think you will add them as SVG and then generate the PNGs so this problem will disappear I guess. Only for the Comment-button I haven't created a symbolic one. Thanks!

Sorry @Sigma1912 @BsAtHome @zz912 that I create this in favor of reviewing your PRs, but these changes are mostly from march and I will get this done to avoid conflicts especially when there will be some activity in the LinuxCNC meeting in Oslo this weekend.

PeterMue · 2025-07-03T07:13:47Z

I'll take a look

zz912 · 2025-07-03T14:16:35Z

Hi,

I tested 3.5.1 and here are my findings:

is possible make tooltips for continous buttons? Not everyone will understand the symbol, so a tooltip might help.
The same "back change" symbol and the same "back to main menu" symbol are close together. Would it be possible to differentiate them at least a little?
Calculator in tooltable is good, but it put numbers with decimal comma. After Apply it crashes.

hansu · 2025-07-03T18:00:52Z

Hi,

I tested 3.5.1 and here are my findings:

1. is possible make tooltips for continous buttons? Not everyone will understand the symbol, so a tooltip might help.

Sure.

2. The same "back change" symbol and the same "back to main menu" symbol are close together. Would it be possible to differentiate them at least a little?

I tried but it was rejected by the community. I didn't remember the issue or thread, but I kept the images in the latest release: f3a8136

3. Calculator in tooltable is good, but it put numbers with decimal comma. After Apply it crashes.

Oh, I didn't test that. The everlasting problems with the localization in the tool table...
@Sigma1912 We decided to drop localization in the tool table, but the calculator still does. For the offsetpage it fits but here not.

hansu · 2025-07-03T18:19:20Z

@PeterMue Please also for the button that came via #3496. Thanks!

Sigma1912 · 2025-07-03T19:47:27Z

We decided to drop localization in the tool table, but the calculator still does. For the offsetpage it fits but here not.

I cannot test this right now but I think replacing this line:

                store[row][col] = locale.format_string("%11.4f", value)

https://github.com/LinuxCNC/linuxcnc/blob/master/src/emc/usr_intf/gmoccapy/gmoccapy.py#L1992

with this

                store[row][col] = ("%11.4f" % value)

should do it.

hansu · 2025-07-03T20:28:26Z

           store[row][col] = ("%11.4f" % value)

That would bee to easy 😄. But yeah, almost that 😉 (4bcfd3e). Thanks.

zz912 · 2025-07-03T20:29:34Z

We decided to drop localization in the tool table, but the calculator still does. For the offsetpage it fits but here not.

I would rather disable localization in the calculator.
Decimal comma makes only problems in technical software.

hansu · 2025-07-03T20:36:40Z

I would rather disable localization in the calculator.
Decimal comma makes only problems in technical software.

Yes, but it should be possibly to use the comma key on the keyboard as decimal separator.
I suggest to show only the dot in the calculator and treat comma and dot as decimal separators.

zz912 · 2025-07-03T20:43:49Z

I would rather disable localization in the calculator.
Decimal comma makes only problems in technical software.

Yes, but it should be possibly to use the comma key on the keyboard as decimal separator. I suggest to show only the dot in the calculator and treat comma and dot as decimal separators.

It was Norberts conditions.

Maybe this will help:

linuxcnc/lib/python/gladevcp/tooledit_widget.py

Line 479 in 4bcfd3e

self.model[path][col] = f"{float(new_text.replace(',', '.')):10.4f}"

hansu · 2025-07-03T20:52:17Z

Hmm but weird that it crashes though.

zz912 · 2025-07-03T21:02:50Z

It crashed here:

linuxcnc/lib/python/gladevcp/tooledit_widget.py

Line 354 in 4bcfd3e

line = line + "%s%s "%(KEYWORDS[num], float(test))

In save function:

linuxcnc/lib/python/gladevcp/tooledit_widget.py

Line 320 in 4bcfd3e

def save(self,widget):

Not in function :

linuxcnc/lib/python/gladevcp/tooledit_widget.py

Line 463 in 4bcfd3e

def col_editted(self, widget, path, new_text, col, filter):

The col_editted function was probably skipped by the calculator.

Sigma1912 · 2025-07-04T07:32:35Z

Yes, but it should be possibly to use the comma key on the keyboard as decimal separator.

We could have the calculator widget accept ',' as well as '.' by converting all commas to dots:

Changing this

        temp = temp.replace(",", ".")

https://github.com/Sigma1912/linuxcnc/blob/master/lib/python/gladevcp/calculatorwidget.py#L174

to this:

        temp = self.eval_string.strip( " " ).replace("Pi", "math.pi")
        temp = temp.replace(",", ".")

Sigma1912 · 2025-07-04T07:49:20Z

An other option would be to do it in the signal handler function similar to what is done in 'col_editted':

Changing this

            else:
                store[row][col] = f"{value:11.4f}"

https://github.com/Sigma1912/linuxcnc/blob/master/src/emc/usr_intf/gmoccapy/gmoccapy.py#L1992

to this

            else:
                value = float(str(value).replace(",", "."))
                store[row][col] = f"{value:11.4f}"

Sigma1912 · 2025-07-04T07:56:19Z

I think I would prefer the second option thus not allowing mixing comma AND dots in the calculator but either commas OR dots depdending on the locale set.
Also it wouldn't disable the calculator widget from handing back float values according to locale setting.

The col_editted function was probably skipped by the calculator.

Yes, the calculator uses the 'editing-started' signal which is emitted as soon as a cell is in editing mode while the 'edited' signal is emitted after the cell has been edited using the keyboard.

zz912 · 2025-07-04T11:54:41Z

[Sigma1912]

I think I would prefer the second option thus not allowing mixing comma AND dots in the calculator but either commas OR dots depdending on the locale set.
Also it wouldn't disable the calculator widget from handing back float values according to locale setting.

I would say that history is repeating itself here.
Please see:
#2966 (comment)

I also preferred to use only comma, or only dot.
But then we found out that somewhere decimal dot is necessary.
For example, using decimal comma in g-code is nonsense.
Using a decimal comma in DRO is ugly.
I concluded from the above discussion:

Use decimal point in all localizations.
Make inputs resistant decimal comma.

If we could agree on this, we could make it a rule and list it here:
http://linuxcnc.org/docs/devel/html/gui/gui-dev-reference.html

Sigma1912 · 2025-07-04T13:40:01Z

@zz912
So are you saying that you would prefer the calculator to hand back only values with decimal point (ie option 1)?
That's also fine with me as I live in decimal dot land.

zz912 · 2025-07-04T13:46:24Z

So are you saying that you would prefer the calculator to hand back only values with decimal point (ie option 1)?

Yes

That's also fine with me as I live in decimal dot land.

You are lucky man.

Sigma1912 · 2025-07-04T14:19:42Z

Could you test this please?
In 'lib/python/gladevcp/calculator_widget.py' around line 210 change this:

        except:
            b = "Error"
        self.entry.set_text( b )
        self.entry.set_position(len(self.eval_string))

to this:

        except:
            b = "Error"
        b = b.replace(",", ".")
        self.entry.set_text( b )
        self.entry.set_position(len(self.eval_string))

This should have the advantage of only accepting the decimal symbol set in the locale (ie dot OR comma but not both) and then hand back values with decimal dot.

zz912 · 2025-07-04T14:29:16Z

I'll look at it tonight.

zz912 · 2025-07-04T20:08:27Z

It did not help.

I tested it with print command:

        except:
            b = "Error"
        b = b.replace(",", ".")
        self.entry.set_text( b )
        print(b)
        self.entry.set_position(len(self.eval_string))

b was with decimal point, but in table cell was comma.

It is weird.

If you want to test other localizations, here is the manual:
#2966 (comment)

hansu · 2025-07-04T21:18:43Z

That's also fine with me as I live in decimal dot land.

Where do you live? Because your name sounds German... Or just a German living abroad? 🤔

Sigma1912 · 2025-07-05T04:33:50Z

I'm in Switzerland

hansu · 2025-07-05T06:11:15Z

Ah, didn't know that they use dots.

Sigma1912 · 2025-07-05T09:19:51Z

Ah, didn't know that they use dots.

I think we are about the only ones in europe beside the irish and the brits

I notice that the offset page uses decimal comma format, is that intended? Seems somewhat odd to use decimal dot in the dro and the tool table but decimal comma in the offset table:

Wouldn`t it be preferable to have the decimal dot format everywhere in Gmoccapy?

hansu · 2025-07-05T09:37:25Z

I notice that the offset page uses decimal comma format, is that intended?

I think this was originally intended by Norbert, like the tool table also used localization.
But now as we agreed to use dots in the tool table and tool information, we should use dots everywhere in Gmoccapy.

zz912 · 2025-07-05T09:46:00Z

Wouldn`t it be preferable to have the decimal dot format everywhere in Gmoccapy?

Yes, yes, yes. My opinion. But don't forget to accept decimal comma in inputs. The best is plain:
.replace(",",` ".")

hansu · 2025-07-09T06:57:42Z

Almost all have completely misunderstood this PR :'(
It should have been a discussion about the features I presented, not about any new ones.
It's now hard to follow the topics itself.

So please continue for the several topics that arose from this in the dedicated topics:

Offset page: #3513
Tool table: #3509
Calculator - localization: #3500

hansu · 2026-01-05T21:48:07Z

I would suggest to merge this like it is now and do the localization stuff for the calculator and so on in the next gmoccapy version. What do you think?

zz912 · 2026-01-05T21:50:07Z

No problem for me.

gmoccapy · 2026-01-06T07:25:47Z

I do agree to push this one now and discuss the comma dot stuff later.
I vote for the replace solution, so leaving the cell can change any number to dot separation and we allow at the same time the use of commy on the keyboard.
IMHO normally Noone will use the coma as thousand separator

zz912 · 2026-01-06T08:03:24Z

Issue for "Calculator widget - localization - decimal separator" is made:
#3686

hal_glib --> common/hal_glib, see 333579a

…ent button box

because this does not reflect the spindle speed programmed with the S word. It shows the current resp. commanded speed.

* feat: new icons for gmoccapy * calculator_open * chevron_down * edit_undo * edit_redo * split_view * comment * jog_continuous * feat: add material icons "add", "delete"

…unused format template

hansu added the affects gmoccapy label Jul 3, 2025

hansu force-pushed the gmoccapy-3-5-1 branch from 9a99bfd to 1d6c46c Compare July 3, 2025 18:17

hansu force-pushed the gmoccapy-3-5-1 branch from 1d6c46c to 5b3d243 Compare July 3, 2025 20:30

zz912 mentioned this pull request Jul 5, 2025

Document - GUI development - added Localization requirements #3499

Merged

hansu mentioned this pull request Jul 9, 2025

Gmoccapy offset page - buttons #3513

Open

hansu force-pushed the gmoccapy-3-5-1 branch from 23a2b62 to 284d829 Compare July 9, 2025 07:31

hansu mentioned this pull request Oct 20, 2025

gmoccapy: allow use of TAB key in the tooledit widget #3586

Merged

hansu force-pushed the gmoccapy-3-5-1 branch 3 times, most recently from 9eebc04 to 7f72caa Compare January 5, 2026 21:20

gmoccapy: re-arrange search and replace bar

de1f182

hansu force-pushed the gmoccapy-3-5-1 branch from 7f72caa to 26b7df8 Compare January 7, 2026 12:03

hansu and others added 14 commits January 7, 2026 14:51

gmoccapy: fix G-code properties due to change in hal_glib

d6a1f3e

hal_glib --> common/hal_glib, see 333579a

gmoccapy: add comment/uncomment button for G-code edit + keybinding

2bcf69a

gmoccapy: bigger MDI enter button (with CSS)

97d256c

gmoccapy: add icon for continuous jog, change alignment of jog increm…

a813f28

…ent button box

gmoccapy: add setting for jog button size + right pane width

c2ae5d1

gmoccapy: add button "refresh dir"

64e994a

gmoccapy: shorten text for macro parameter names

2490778

gmoccapy: add button to show preview beside edit window (split view)

f873fd8

gmoccapy: remove S prefix for spindle display

f206df0

because this does not reflect the spindle speed programmed with the S word. It shows the current resp. commanded speed.

gmoccapy: style velocity info box

c49ee45

feat: new icons for gmoccapy (#9)

2b0f7a9

* feat: new icons for gmoccapy * calculator_open * chevron_down * edit_undo * edit_redo * split_view * comment * jog_continuous * feat: add material icons "add", "delete"

Offsetpage_widget: force display of unlocalized float values, remove …

ec18c84

…unused format template

gmoccapy: optimize toolinfo grid

99c49e6

gmoccapy: add settings for usage of calculator in toolpage/offsetpage

77600ab

hansu force-pushed the gmoccapy-3-5-1 branch from 26b7df8 to 5475ad4 Compare January 7, 2026 17:51

gmoccapy_3_5_1: update release notes and docs, set version number

5475ad4

hansu merged commit 6f877af into LinuxCNC:master Jan 7, 2026
13 checks passed

Gmoccapy 3.5.1 #3494

Gmoccapy 3.5.1 #3494

Conversation

hansu commented Jul 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

PeterMue commented Jul 3, 2025

Uh oh!

zz912 commented Jul 3, 2025

Uh oh!

hansu commented Jul 3, 2025

Uh oh!

hansu commented Jul 3, 2025

Uh oh!

Sigma1912 commented Jul 3, 2025

Uh oh!

hansu commented Jul 3, 2025

Uh oh!

zz912 commented Jul 3, 2025

Uh oh!

hansu commented Jul 3, 2025

Uh oh!

zz912 commented Jul 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hansu commented Jul 3, 2025

Uh oh!

zz912 commented Jul 3, 2025

Uh oh!

Sigma1912 commented Jul 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Sigma1912 commented Jul 4, 2025

Uh oh!

Sigma1912 commented Jul 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

zz912 commented Jul 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Sigma1912 commented Jul 4, 2025

Uh oh!

zz912 commented Jul 4, 2025

Uh oh!

Sigma1912 commented Jul 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

zz912 commented Jul 4, 2025

Uh oh!

zz912 commented Jul 4, 2025

Uh oh!

hansu commented Jul 4, 2025

Uh oh!

Sigma1912 commented Jul 5, 2025

Uh oh!

hansu commented Jul 5, 2025

Uh oh!

Sigma1912 commented Jul 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hansu commented Jul 5, 2025

Uh oh!

zz912 commented Jul 5, 2025

Uh oh!

hansu commented Jul 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hansu commented Jan 5, 2026

Uh oh!

zz912 commented Jan 5, 2026

Uh oh!

gmoccapy commented Jan 6, 2026

Uh oh!

zz912 commented Jan 6, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

hansu commented Jul 3, 2025 •

edited

Loading

zz912 commented Jul 3, 2025 •

edited

Loading

Sigma1912 commented Jul 4, 2025 •

edited

Loading

Sigma1912 commented Jul 4, 2025 •

edited

Loading

zz912 commented Jul 4, 2025 •

edited

Loading

Sigma1912 commented Jul 4, 2025 •

edited

Loading

Sigma1912 commented Jul 5, 2025 •

edited

Loading

hansu commented Jul 9, 2025 •

edited

Loading