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

Currently doesn't work on macOS, or without STK. #6

Open
3 tasks
techdragon opened this issue Dec 3, 2021 · 3 comments
Open
3 tasks

Currently doesn't work on macOS, or without STK. #6

techdragon opened this issue Dec 3, 2021 · 3 comments

Comments

@techdragon
Copy link

  • Automatically imports STK stuff that is Windows only and also appears to not be free/open software, but since its optional this isn't required.
  • Windows specific paths break on macOS (will also break on Linux)
  • Also kinda sensitive to the underlying TCL/TK library in use by the python interpreter running the script. Very broken on macOS without a recent version.

I've got a few fixes for some of these, but the easiest way will be to package this up with a proper dependency management file (I've already got the pyproject.toml from setting this up with poetry on my Mac.) but I don't want to shove a big pull request where I package it, and make a big part of the codebase optional without asking if that's going to be accepted first.

@sammmlow
Copy link
Owner

sammmlow commented Jan 5, 2022

Thanks for bringing it up! Feel free to make a pull request, and we can review the changes - it'll be very much appreciated since I don't have a Mac or Linux terminal to test it out on; I've had a few big issues before with other Mac and Linux users, especially with imports of STK-related things or the comtypes library (STK app works only with Windows).

Unfortunately that's one shortcoming of TKinter - the GUI isn't agnostic to the user's display settings. Sometimes the widgets take up too much or too little room. Currently the only thing that's been done is to sample the DPI of the user, and scale it accordingly for the matplotlib plots. At present I haven't yet thought of a way to move forward with the clunky GUI though.

@sammmlow
Copy link
Owner

sammmlow commented Jan 6, 2022

Could you also upload screenshots of the GUI or any error messages that you observed?

@kerel-fs
Copy link
Contributor

Just submitted a patch solving these issues in Pull Request #9 doing minimal changes.

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

3 participants