Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run on Windows? #27

Open
K4KDR opened this issue Oct 5, 2018 · 5 comments
Open

Run on Windows? #27

K4KDR opened this issue Oct 5, 2018 · 5 comments

Comments

@K4KDR
Copy link
Contributor

K4KDR commented Oct 5, 2018

On Windows 7 (64-bit), after installing the latest Python-3 (Python 3.7) and all the modules that were reported missing when attempting to run:

python minxss_beacon_decoder.py

Here is the output (followed by commands showing the currently running commit version and python version installed):

C:\Users\CSI\Apps\MinXSS_Beacon_Decoder>python minxss_beacon_decoder.py

libpng warning: iCCP: known incorrect sRGB profile
Traceback (most recent call last):
  File "C:\Users\CSI\Apps\Python37\lib\configparser.py", line 788, in get
    value = d[option]
  File "C:\Users\CSI\Apps\Python37\lib\collections\__init__.py", line 914, in __getitem__
    return self.__missing__(key)            # support subclasses that define __missing__
  File "C:\Users\CSI\Apps\Python37\lib\collections\__init__.py", line 906, in __missing__
    raise KeyError(key)
KeyError: 'serial_port'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "minxss_beacon_decoder.py", line 634, in <module>
    main()
  File "minxss_beacon_decoder.py", line 628, in main
    mainWin = MainWindow()
  File "minxss_beacon_decoder.py", line 39, in __init__
    self.setup_last_used_settings()
  File "minxss_beacon_decoder.py", line 151, in setup_last_used_settings
    self.set_instance_variables_from_config(parser)
  File "minxss_beacon_decoder.py", line 171, in set_instance_variables_from_config
    self.comboBox_serialPort.insertItem(0, parser.get('input_properties', 'serial_port'))
  File "C:\Users\CSI\Apps\Python37\lib\configparser.py", line 791, in get
    raise NoOptionError(option, section)
configparser.NoOptionError: No option 'serial_port' in section: 'input_properties'

======

C:\Users\CSI\Apps\MinXSS_Beacon_Decoder>git show --oneline -s

9998410 (HEAD -> master, origin/master, origin/HEAD) Auto updates output files when user changes ground station parameters

======

C:\Users\CSI\Apps\MinXSS_Beacon_Decoder>python --version

Python 3.7.0

======

Kindly advise if there are additional items required to run the current code on a Windows system.

Thank you!

@jmason86
Copy link
Owner

jmason86 commented Oct 5, 2018

It looks like the serial_port is missing from the input_properties.cfg. Could you also post the log/ file and the input_properties.cfg file?

@K4KDR
Copy link
Contributor Author

K4KDR commented Oct 5, 2018

Ok, that feedback identified the problem and solved a mystery. When I attempted to run the app initially (and got these errors), I noted that there was NO input_properties.cfg file present in the folder where I was running the app from. I copied over my working .cfg file from the linux computer but the errors were exactly the same.

Following your reply above, I searched the ENTIRE computer for any instances of input_properties.cfg and found the old MinXSS-1 version in:

C:\Users\{username}\MinXSS_Beacon_Decoder

... when I renamed that file to in effect remove it from the computer and re-ran the current app, a properly-formatted input_properties.cfg file was created in:

C:\Users\{username}\MinXSS_Beacon_Decoder

... and the application GUI launched successfully.

Would it be possible (and probably recommended) for the .cfg file to be created in whatever folder the user is running the application from & not hard-coded to a specific location?

@K4KDR
Copy link
Contributor Author

K4KDR commented Oct 5, 2018

Additional: With the app now running successfully on Windows-7 (64-bit), recorded sample audio was decoded successfully:

2018-10-05--minxss-2--windows-test

@K4KDR
Copy link
Contributor Author

K4KDR commented Oct 5, 2018

Additional: Please note that the folder path used on the Windows platform for OUTPUT and LOG files also appears to be hard-coded to the

C:\Users\{username}\MinXSS_Beacon_Decoder

location.

@jmason86
Copy link
Owner

jmason86 commented Oct 5, 2018

Yes it is sort of hard coded in there to always be in that specific location. That came about because the built application will actually fail to even launch if it's set to the current directory (this is what it was originally). I think it has to do with permissions in the directory that the application runs or something.

There's got to be a way to have it both ways... I keep trying to think of a way to add/use a user specified working directory path to the Input/Options but keep running into issues that have already caused crashes in the past.

I'll keep thinking about this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants