Mesa Configuration Tool

HansU
Offline
Platinum Member

17 Sep 2022 17:00 - 17 Sep 2022 17:02 #252179 by HansU

Replied by HansU on topic Mesa Configuration Tool

Some other annotations:

When I select a firmware like 5i25_7i85sx2 - shouldn't it be generally possible to select two daughter cards?
And these drop down menus are empty for me:
Backups - This could be understood as backup of the firmware because of the layout. Maybe change the text to "Enable Configuration Backups"

It creates some non-existing pins the the HAL file:

hm2_5i25.0.7i85s.0.1.analogena
hm2_5i25.0.7i85s.0.1.analogout0-scalemax
hm2_5i25.0.7i85s.0.1.analogout0-minlim   
hm2_5i25.0.7i85s.0.1.analogout0-maxlim  
hm2_5i25.0.7i85s.0.1.analogout0

In Axes I have to provide values for the PID controller. I think the config generation shouldn't fail I don't provide PID parameters in case I don't want to use a PID controller.

Last edit: 17 Sep 2022 17:02 by HansU.

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

HansU
Offline
Platinum Member

17 Sep 2022 19:42 - 18 Sep 2022 13:32 #252202 by HansU

Replied by HansU on topic Mesa Configuration Tool

And the error message could be improved to be more helpful.
I did some changes in the INI file, and mesact came up with:

File <string> line 1, in function <module>
Error: name 'YES' is not defined
["NameError: name 'YES' is not defined\n"]

It would be nice if gives me a line number e.g.

Furthermore in [DISPLAY] isn't "MIN_VELOCITY" kind of deprecated and should be replaced by" MIN_LINEAR_VELOCITY" ?

Edit:

The problem was that I inserted `HOME_IGNORE_LIMITS = YES` but the tool uses "True" for that.
According to the docs it should be YES/NO.
The same for "HOME_USE_INDEX"
"HOME_IS_SHARED" has to be 0/1 according to the docs.
Maybe LinuxCNC accepts True/False as well, but I think you tool should at least accept the values from the docs .

Last edit: 18 Sep 2022 13:32 by HansU.

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

HansU
Offline
Platinum Member

18 Sep 2022 12:39 #252256 by HansU

Replied by HansU on topic Mesa Configuration Tool

I discovered another bug:
1. open wrong INi file (one that has not been created with that tool)
2. when opening the correct INI file, it fails with the error message from my last post.

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

JT
Away
Administrator

18 Sep 2022 12:57 #252260 by JT

Replied by JT on topic Mesa Configuration Tool

I'll take a look at these as soon as I'm back up and running.

JT

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

HansU
Offline
Platinum Member

18 Sep 2022 13:30 - 18 Sep 2022 13:33 #252263 by HansU

Replied by HansU on topic Mesa Configuration Tool

Commit 2856f614c7344279b0e23974c6ddcdba04794c81 "add output invert for 7i96s"
produces the following error message with my config:

File machine.py line 122, in function daughterCardChanged
Error: name 'MAIN_BOARDS' is not defined
["NameError: name 'MAIN_BOARDS' is not defined\n"]

Furthermore it would be good if unknown key value pairs (like "BACKLASH = 0.01") are not removed when generating the config. Maybe you can add a special section for that? Or remember the lines which didn't correspond to a supported value and print them when generating? This would also allow to keep comments.

Last edit: 18 Sep 2022 13:33 by HansU.

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

JT
Away
Administrator

18 Sep 2022 15:35 - 18 Sep 2022 16:04 #252269 by JT

Replied by JT on topic Mesa Configuration Tool

HansU wrote:
A comment on the commit "fix checking for linuxcnc version I hope"
If I set the localization to en, then it works for me. But with German locale if didn't work because your query gives this:
$ apt-cache policy linuxcnc-uspace
linuxcnc-uspace:
    Installiert: 1:2.8.3
Not sure how to get this fixed multilangual.

Good news I have Debian 11.5 installed on my dev PC but can't change the display resolution so on to Debian 12 in the morning.

I have an idea to test when I get my dev PC back up and going.

JT

Last edit: 18 Sep 2022 16:04 by JT.

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

JT
Away
Administrator

18 Sep 2022 21:08 #252299 by JT

Replied by JT on topic Mesa Configuration Tool

HansU wrote:

Commit 2856f614c7344279b0e23974c6ddcdba04794c81 "add output invert for 7i96s"
produces the following error message with my config:

File machine.py line 122, in function daughterCardChanged
Error: name 'MAIN_BOARDS' is not defined
["NameError: name 'MAIN_BOARDS' is not defined\n"]



Furthermore it would be good if unknown key value pairs (like "BACKLASH = 0.01") are not removed when generating the config. Maybe you can add a special section for that? Or remember the lines which didn't correspond to a supported value and print them when generating? This would also allow to keep comments.

I just pushed a fix for the missing variables.

What section would key value pairs be in that are not LinuxCNC specific? BACKLASH is a joint specification which could be added to the tool easy enough.

JT

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

JT
Away
Administrator

18 Sep 2022 22:01 #252305 by JT

Replied by JT on topic Mesa Configuration Tool

HansU wrote:
A comment on the commit "fix checking for linuxcnc version I hope"
If I set the localization to en, then it works for me. But with German locale if didn't work because your query gives this:
$ apt-cache policy linuxcnc-uspace
linuxcnc-uspace:
    Installiert: 1:2.8.3


Not sure how to get this fixed multilangual.

I changed the way I check for LinuxCNC version that should be language agnostic so if you can test that on your PC with the German locale and let me know if you get the correct LinuxCNC version.

JT

The following user(s) said Thank You: tommylight

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

JT
Away
Administrator

18 Sep 2022 22:28 #252306 by JT

Replied by JT on topic Mesa Configuration Tool

HansU wrote:
Some other annotations:
When I select a firmware like 5i25_7i85sx2 - shouldn't it be generally possible to select two daughter cards?

And these drop down menus are empty for me:
Backups - This could be understood as backup of the firmware because of the layout. Maybe change the text to "Enable Configuration Backups"
It creates some non-existing pins the the HAL file:
hm2_5i25.0.7i85s.0.1.analogena
hm2_5i25.0.7i85s.0.1.analogout0-scalemax
hm2_5i25.0.7i85s.0.1.analogout0-minlim   
hm2_5i25.0.7i85s.0.1.analogout0-maxlim  
hm2_5i25.0.7i85s.0.1.analogout0
In Axes I have to provide values for the PID controller. I think the config generation shouldn't fail I don't provide PID parameters in case I don't want to use a PID controller.

At this time I only have the code base for one daughter card... in the future I'll work on two daughter cards.

Also only some boards are coded to allow reducing the number of stepgens etc.

Backup text changed to be clearer what the function is.

I'll have to see why those pins are created for the 7i85s, can you attach your ini file?

Mesa cards need the PID so it's required.

JT

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

HansU
Offline
Platinum Member

19 Sep 2022 11:36 #252335 by HansU

Replied by HansU on topic Mesa Configuration Tool

JT wrote:
Backup text changed to be clearer what the function is.

:+1:

JT wrote:
I'll have to see why those pins are created for the 7i85s, can you attach your ini file?

It still complains about when opening the config file: `Error: name 'MAIN_BOARDS' is not defined`

JT wrote:
Mesa cards need the PID so it's required.

For what they need a PID for a simple stepper config? I removed the PID component and it still works.
See attached files.

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

Time to create page: 0.239 seconds

Mesa Configuration Tool

Attachments:

Attachments: