Config will not open after Buildbot2 upgade from 2.9.4 to 2.10

  • rodw
  • rodw's Avatar Topic Author
  • Offline
  • Platinum Member
  • Platinum Member
More
22 Mar 2025 05:39 #324520 by rodw
Can Phill or Snowgoer have a look at this issue? Sorry I thought it was wider than QTPlasmac but further testing shows it only affects QTP.
forum.linuxcnc.org/9-installing-linuxcnc...-on-buildbot2#324519
This is on a SIM only and I need to upgrade a live config
Thanks

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

  • rodw
  • rodw's Avatar Topic Author
  • Offline
  • Platinum Member
  • Platinum Member
More
22 Mar 2025 11:33 #324534 by rodw
I Initially thought it was due to an incorrect .ini file version but I eliminated that as the fault
 
Is this error message generated by QTplasmac?
It is not documented anywhere I could see. In any case it should not continue to appear.
 
Attachments:

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

  • tommylight
  • tommylight's Avatar
  • Away
  • Moderator
  • Moderator
More
22 Mar 2025 12:08 #324540 by tommylight
Should we delete the other topic, that i already moved? :)

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

  • rodw
  • rodw's Avatar Topic Author
  • Offline
  • Platinum Member
  • Platinum Member
More
22 Mar 2025 12:23 #324541 by rodw
Yeh, I guess so.

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

More
22 Mar 2025 19:31 #324582 by snowgoer540

Can Phill or Snowgoer have a look at this issue?


That particular message is to let you know your config has been update to 001.017, and that you need to restart for things to take effect.

If you are seeing it repeatedly, seeing you had a permissions issue in another post, I’d guess you still have some permissions issue(s) going on.

Otherwise if you wanted to zip your config and post it here, I’ll try to make time to take a look.

In particular, that update is looking for 'nc_files/plasmac/m_files' in your USER_M_PATH, to be able to see the Mxxx files. It’s not seeing it so it’s trying to update the ini file to fix it.

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

More
22 Mar 2025 19:32 #324583 by snowgoer540

Snowgoer


Greg … just in case you somehow missed it after all these years :)

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

  • rodw
  • rodw's Avatar Topic Author
  • Offline
  • Platinum Member
  • Platinum Member
More
22 Mar 2025 21:14 #324584 by rodw
Thanks Greg, I've beem away for years and find a bug in 5 minutes 
In this case the the config to upgrade was nothing more than a sim!
Issues:
  1. The m_files folder was in ~/linuxcnc/nc_files/examples/plasmac, copied it into ~/linuxcnc/nc_files
  2. The files in the m_files folder under examples were owned by root and the user did not have permissions to the M codes
  3. The INI M_CODE variable in the ini file was not updated in qtplasmac_metric_ini.inc. I suspect this was because its not the top level ini file (I'm not sure if you actually write to that folder)
  4. The error message should refer to plasmac so the user knows its specific to Plasmac. I had to deduce that by checking other configs
  5. The version number reported does not reconcile with the version details displayed on startup 2.10.0-pre-4165-xxxxxxxxxxxxxxxx
  6. Let me introduce you to Python try/except error handling www.w3schools.com/python/python_try_except.asp This would ensure the user is less bewildered when it  fails. There is nothing displayed on the screen when it fails
  7. Perhaps the message could be a little more specific eg "updating qtplasmac mcode folder"
  8. Perhaps the upgrade message could also be printed on the console. Displaying Top right when console is in bottom left meant I did not even see the error the first couple of times!
I might add Joco is also in trouble over this for adding these gcodes!   

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

  • tommylight
  • tommylight's Avatar
  • Away
  • Moderator
  • Moderator
More
22 Mar 2025 22:22 #324590 by tommylight

Snowgoer


Greg … just in case you somehow missed it after all these years :)


Nope, i will continue with Snowwy ! :)
The following user(s) said Thank You: rodw, snowgoer540

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

More
23 Mar 2025 01:51 #324601 by snowgoer540

Thanks Greg, I've beem away for years and find a bug in 5 minutes 


You've been doing a lot of posting over the years for someone who has been away for years ...

I'd hold the back patting, I think you have something else going on ...

The m_files folder was in ~/linuxcnc/nc_files/examples/plasmac


Perfect that's where they're supposed to be:

github.com/LinuxCNC/linuxcnc/commit/d4bf...8829a665f015f1982562

Note that this ends up being nc_files/examples/plasmac/m_files on a package install.

The files in the m_files folder under examples were owned by root and the user did not have permissions to the M codes


That's possible, I don't do much with package installs, but it shouldn't impact the end user either way.

I installed the latest package on buildbot2 for bookworm a few minutes ago, it opened the qtplasmac_l_metric simulation without issue, no update messages, etc.

The INI M_CODE variable in the ini file was not updated in qtplasmac_metric_ini.inc. I suspect this was because its not the top level ini file (I'm not sure if you actually write to that folder)


This was fixed on Sept 6, 2024. Commit notes:
qtplasmac: fix m_code pathing
Fixes the m_code path for a package install sim.
Allows update 001-017 to be applied correctly for configs that were build prior to then.

github.com/LinuxCNC/linuxcnc/commit/da67...c33db02daefaa2460e18

The error message should refer to plasmac so the user knows its specific to Plasmac. I had to deduce that by checking other configs


Did you think it referred to QtDragonHD, or Axis? ;)

I suppose it could say something to the effect of "QtPlasmaC configuration changes mandated at V2.10-001.017 were applied". As a developer it's not always easy to know how the end user will interpret a message. Since January 23, 2024 you are the very first to complain.

The version number reported does not reconcile with the version details displayed on startup 2.10.0-pre-4165-xxxxxxxxxxxxxxxx


It's not supposed to. The main takeaway from the message is you have to restart QtPlasmaC. Similar to this case, the screenshot helps us with figuring out where the issue might lie.

Let me introduce you to Python try/except error handling www.w3schools.com/python/python_try_except.asp This would ensure the user is less bewildered when it  fails. There is nothing displayed on the screen when it fails


I am quite aware of try/except error handling. It's pompous comments like these coupled with the shameless back patting that usually serve to make me sorry I tried to help you in the first place.

Both functions are wrapped in try/except, as interacting with the filesystem sometimes results in unexpected behavior.

There must be something going on with your OS/Installation. Simulations work fine on version V2.10-008.056 for a Package install and V2.10-008.061 for RIP. No update messages.

Perhaps the message could be a little more specific eg "updating qtplasmac mcode folder"


It serves the purpose as is.

Perhaps the upgrade message could also be printed on the console. Displaying Top right when console is in bottom left meant I did not even see the error the first couple of times!


Click the Statistics tab, there's a MACHINE LOG frame, it'll be there. machine_log_{date/time}.txt in the config folder also reflects the log on the Statistics tab. It's also in the "update_log.txt" file in the config folder.

I might add Joco is also in trouble over this for adding these gcodes! 


Ok.

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

  • rodw
  • rodw's Avatar Topic Author
  • Offline
  • Platinum Member
  • Platinum Member
More
23 Mar 2025 03:11 #324606 by rodw

In particular, that update is looking for 'nc_files/plasmac/m_files' in your USER_M_PATH, to be able to see the Mxxx files. It’s not seeing it so it’s trying to update the ini file to fix it.

Perfect that's where they're supposed to be:


Your previously advised location was different. What is correct?

I installed the latest package on buildbot2 for bookworm a few minutes ago, it opened the qtplasmac_l_metric simulation without issue, no update messages, etc.

That would be expected. You would need to install on 2.94 then upgrade to 2.10 to replicate the issue. Confirmed behavoiur on 2 different PC's

This was fixed on Sept 6, 2024. Commit notes:

This was not fixed for sims because I installed packages dated Feb 2025 from the buildbot. Bug remains. 

Did you think it referred to QtDragonHD, or Axis? ;)

I suppose it could say something to the effect of "QtPlasmaC configuration changes mandated at V2.10-001.017 were applied". As a developer it's not always easy to know how the end user will interpret a message. Since January 23, 2024 you are the very first to complain.


I had no idea as Axis, QTdragon and QTplasmac.sim configs were installed on my PC.
Clearer messaging would be nice 
I had a specific reason to test upgrade from 2.9.4 to 2.10.

There must be something going on with your OS/Installation. Simulations work fine on version V2.10-008.056 for a Package install and V2.10-008.061 for RIP. No update messages.

Sadly, the buildbot must be stuck again if it is that far behind for a package install done yesterday.
You would need to build for 2.9.4, install the sim, then open the config in 2.10 to replicate this error. Installing on 2.10 proves nothing as no upgrade will be triggered.

I can't use RIP given my customer is 1000 miles away and I want him to upgrade to 2.10 becasue you are no longer supporting 2.9

PC is running Trixie with kernel 6.12. But it also happened on a Bookworm PC. I think Debian version is irrelavent. (Trixie is only a couple of months away and 2.9.4 will be in it)

Click the Statistics tab, there's a MACHINE LOG frame, it'll be there. machine_log_{date/time}.txt in the config folder also reflects the log on the Statistics tab. It's also in the "update_log.txt" file in the config folder.

There is no erors in any of the MACHINE logs. There is no information in the update logs. We need better error reporting.

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

Moderators: snowgoer540
Time to create page: 0.100 seconds
Powered by Kunena Forum