Configurator Error

More
20 Dec 2019 16:26 - 20 Dec 2019 16:27 #152955 by AgentWD40
Configurator Error was created by AgentWD40
I just wanted to report an error I ran into with the configurator.py. I'm trying to get an updated plasmac simulation running. This isn't actually connected to my machine, let me know if this is the root of the configurator's problem here.

I'm using linuxcnc 2.9.0-pre0-... from buildbot master.

All I've done for config so far is use the Sample Configurations -> by_machine -> plasmac -> gmoccapy -> imperial_plasmac

This runs fine for the simulator. However I don't have the shapes stuff in the extras tab. So I attempted to run the configurator on this imperial_plasmac.ini

Python throws the following error after hitting upgrade:
Ugrading from v0.3 to v0.6
Traceback (most recent call last):
  File "configurator.py", line 452, in on_create_clicked
    self.upgrade_ini_file(version,display)
  File "configurator.py", line 791, in upgrade_ini_file
    buttons[n][0] = line
IndexError: list index out of range

Also, just an FYI, I noticed that if the user closes the file picker dialog box without picking a file then python throws an error and then freezes. I just wanted to mention it in case you wanted to throw in some error handling there.
Last edit: 20 Dec 2019 16:27 by AgentWD40.

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

More
20 Dec 2019 21:35 #152974 by phillc54
Replied by phillc54 on topic Configurator Error
Thanks for the report.

I have never tried to upgrade a sample config because I only use Run In Place and it doesn't copy the sample config over. I will give it a try.

Error checking is the bane of my life...

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

More
20 Dec 2019 21:48 - 21 Dec 2019 01:11 #152976 by phillc54
Replied by phillc54 on topic Configurator Error

All I've done for config so far is use the Sample Configurations -> by_machine -> plasmac -> gmoccapy -> imperial_plasmac

Would you be able to zip this complete config and post it here?

Edit: I have pushed what I "think" is a fix for these issues.
Last edit: 21 Dec 2019 01:11 by phillc54.

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

More
21 Dec 2019 03:08 #152981 by tommylight
Replied by tommylight on topic Configurator Error

phillc54 wrote:
Error checking is the bane of my life...

A professor of electronics in Germany told a friend of mine:
Electronics is 10% work, 90% troubleshooting that same work !
Luckily i had it in reverse! But then there is reverse engineering of electronics........... spent the day doing just that !

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

More
21 Dec 2019 03:12 #152982 by phillc54
Replied by phillc54 on topic Configurator Error

tommylight wrote: A professor of electronics in Germany told a friend of mine:
Electronics is 10% work, 90% troubleshooting that same work !

I think it might be a similar ratio for software, at least it seems it is for me.
The following user(s) said Thank You: tommylight

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

More
27 Dec 2019 21:51 #153320 by AgentWD40
Replied by AgentWD40 on topic Configurator Error
Heads up Phill, looks like a recent update messed up some of the indention in the configurator.py around line #801.
forum.linuxcnc.org/9-installing-linuxcnc/38051-plasmac

Why does python rely on indentation levels anyway? Never understood that...

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

More
28 Dec 2019 00:22 #153343 by phillc54
Replied by phillc54 on topic Configurator Error

AgentWD40 wrote: Heads up Phill, looks like a recent update messed up some of the indention in the configurator.py around line #801.
forum.linuxcnc.org/9-installing-linuxcnc/38051-plasmac

Thanks Kyle, I have just pushed the fixes.


Why does python rely on indentation levels anyway? Never understood that...

I think it is supposed to be easier to spot errors if it is nice and tidy, but that obviously doesn't help me. :(

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

More
28 Dec 2019 12:28 #153362 by rodw
Replied by rodw on topic Configurator Error

AgentWD40 wrote: Why does python rely on indentation levels anyway? Never understood that...


I think its supposed to save some keystrokes for developers becasue it gets away from {} and if..end
But generally if you open a file in a different editor it breaks the indentation...

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

More
04 Jan 2020 20:29 - 04 Jan 2020 20:31 #154020 by snowgoer540
Replied by snowgoer540 on topic Configurator Error
I think I may have discovered a new error with configurator.py. I found it when trying to reconfigure an existing setup. See attached picture.

To replicate: Run configurator, click "Reconfigure", change radio button for THC Modes. Error results.

Maybe I did something wrong?
Attachments:
Last edit: 04 Jan 2020 20:31 by snowgoer540.

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

More
04 Jan 2020 22:22 #154036 by phillc54
Replied by phillc54 on topic Configurator Error
Should be OK in the latest version.

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

Moderators: phillc54
Time to create page: 0.167 seconds
Powered by Kunena Forum