kliment / Printrun

Pronterface, Pronsole, and Printcore - Pure Python 3d printing host software
GNU General Public License v3.0
2.36k stars 995 forks source link

WIn-build Actions are expired - no downloads of any working builds that newer than 2.0.0rc8 #1295

Closed DaLiV closed 1 year ago

DaLiV commented 1 year ago

wanted to try latest versions for windows

DivingDuck commented 1 year ago

Had you try this run? https://github.com/kliment/Printrun/actions/runs/3682013645

Development versions from actions are only a limited time available and do expire.

DaLiV commented 1 year ago

Failed to execute due ...

Traceback (most recent call last): File "pronterface.py", line 62, in File "printrun\pronterface.py", line 2580, in init File "printrun\pronterface.py", line 186, in init wx._core.wxAssertionError: C++ assertion "strcmp(setlocale(0, 0), "C") == 0" failed at ....\src\common\intl.cpp(1699) in wxLocale::GetInfo(): You probably called setlocale() directly instead of using wxLocale and now there is a mismatch between C/C++ and Windows locale. Things are going to break, please only change locale by creating wxLocale objects to avoid this!

OS: Win11-Home installed Pythons (from MS-Store) : 3.8 ... but think that also not matter, as exe not related on system python-s

DivingDuck commented 1 year ago

Thanks for this info. This this seems to be the wxPython issue and happen to Windows 10 and 11 with some language setting combinations. I have a version where this is solved. You can try this interim version if you want:

https://github.com/DivingDuck/Printrun/actions/runs/3748727767

Scroll to the end of the page and download one of the artifacts. The number at the end of the file identify the python version used for compiling. This is the only difference.

This Wiki page list all changes compared to the latest release candidate: https://github.com/DivingDuck/Printrun/wiki/My-latest-development-version-build-with-github-workflow Feedback is welcome.

DaLiV commented 1 year ago

same for all (3.7-3.10) gui has started but on [connect] - not connecting ... and no problem descriptions - empty traceback.

Connecting...
--- Logging error ---
Traceback (most recent call last):

During handling of the above exception, another exception occurred:
Not connected to printer.
--- Logging error ---
Traceback (most recent call last):
  File "logging\__init__.py", line 1088, in emit
  File "printrun\pronterface.py", line 108, in write
AttributeError: 'NoneType' object has no attribute 'write'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "printrun\printcore.py", line 150, in logError
  File "printrun\pronsole.py", line 297, in logError
  File "logging\__init__.py", line 2049, in error
  File "logging\__init__.py", line 1475, in error
  File "logging\__init__.py", line 1589, in _log
  File "logging\__init__.py", line 1599, in handle
  File "logging\__init__.py", line 1661, in callHandlers
  File "logging\__init__.py", line 954, in handle
  File "logging\__init__.py", line 1093, in emit
  File "logging\__init__.py", line 1006, in handleError
  File "printrun\pronterface.py", line 108, in write
AttributeError: 'NoneType' object has no attribute 'write'

--- Logging error ---
Traceback (most recent call last):
During handling of the above exception, another exception occurred:
During handling of the above exception, another exception occurred:
During handling of the above exception, another exception occurred:
DivingDuck commented 1 year ago

Interesting, you are the the second person reporting this. Can you check the issue below? https://github.com/DivingDuck/Printrun/issues/22 It seems like you have a defect printrunconf.ini. This happen usually if you switch between 1x and 2.x versions.

Please delete this file in: c:\Users\Your_User_Name\ and delete the folder: c:\Users\Your_User_Name\AppData\Local\Printrun\ as described. Then open Pronterface and set the log file to an appropriate file name including the path information as I had shown. You need to have full read/write access to this location. Like: c:\Users\Your_User_Name\Printrun.log

Your_User_Name = your windows user name

DaLiV commented 1 year ago

so ... that was "log file" issue has tried to clean up LocalProfile before - not helped

so part with log helped

to solve / improve : a) default = not to log, especially when no log file set (that must be advisable when software try to do that) b) set default log location in %TEMP%, %LOCALAPPDATA%, %APPDATA% ... not in %USERPROFILE% c) on unwriteable log - alert and open "create LOG file" dialog

DivingDuck commented 1 year ago

Thanks for your feedback, I appreciate that. The original behavior was not to log if there is no logging file specified. I'll take a look on that but I can't promise to change this as we do not have only windows users to consider here. In addition %LOCALAPPDATA%, %APPDATA% are hidden folders and usual users do not to see this locations in Explorer what cause additional problems / issue reports.

Anyway, I see the behavior of Pronterface had change and I can reproduce the issue with your information. There should be at least a check to prevent logging if there is no entry made.

Again, thanks for reporting.

DivingDuck commented 1 year ago

I had the chance to take a quick look on the issue. Pronterface should redirect all logging activities to console output if there is no log path specified. Exact this happen if I run Pronterface directly from source code. Running Ponterface from the executable file will show an error if log path is not set. This happen in the latest artifact builds here as well as in my latest builds local and via my repository. I have no solution for this right now. The work around for now is to set path and filename in log path to an appropriate location where the user have read/write access. Such as: c:\Users\Your_User_Name\Printrun.log (Your_User_Name = your windows user name)

DivingDuck commented 1 year ago

@DaLiV , it took a while to fix that problem as well, but now this issue is solved too with the latest updates we did for the repository here. You don't need to download the files from my repository as it is in sync with my updates. You can test and give us a feedback if you like so. Here is a link to the latest build. https://github.com/kliment/Printrun/actions/runs/4152400998

I will close the issue and mark as completed (https://github.com/kliment/Printrun/commit/e66bbbdeab17dfd6951299d92c4d8dc0cd32bb05)