camUrban / PteraSoftware

Ptera Software is a fast, easy-to-use, and open-source software package for analyzing flapping-wing flight.
MIT License
168 stars 32 forks source link

Create gui #18

Closed Zach10a closed 1 year ago

Zach10a commented 2 years ago

Creation of a very basic GUI

Motivation

Allows greater usability and is useful for those not as technically gifted with python. I am putting this PR in now so that others may help out if they so desire and the final product can be discussed.

Relevant Issues

I intended to create a package installer for this. I will be creating the package installer in a different PR as it is a little big for one feature branch.

Changes

Created a GUI

New Dependencies

PyInstaller >= 5.0.0 in requirements_dev.txt PySide2~=5.15.2.1 in requirements.txt

Change Magnitude

Not release-worthy yet, however once this and the package installer have been merged it will require a release.

Zach10a commented 2 years ago

Travis CI is not working due to issues with Qt. Recommend the following https://stackoverflow.com/questions/56826665/travis-ci-failing-to-build-due-to-pyside2-and-matplotlib

camUrban commented 2 years ago

Hey @Zach10a! Thanks for this awesome pull request and all the hard work you've put in. I apologize for the radio silence these last few days; things have been pretty crazy at work. I should be able to review your request tomorrow and we can move forward with getting it merged!

Zach10a commented 2 years ago

No worries at all, I've been on holiday all week so will get back to it ASAP if you need any more info/changes

Zach10a commented 2 years ago

Fixes #18

Zach10a commented 2 years ago

Hi @camUrban, hope you're doing alright and work isn't too manic. Absolutely not a priority but just a bump in case you've not seen this pop up anywhere. I intend to leave it how it is and not clutter up the commits to fix the Travis CI and fix it in a branch afterwards. I also have the package branch pretty much good to go other than some troubleshooting!

camUrban commented 2 years ago

Hey man! I'm so sorry again about the delay. I'm in the final two weeks of my current job, so I'm working overtime to wrap up several projects. I feel bad because I know how much time you've put into this, but I think I'll need to delay reviewing this request until after I finish. However, on Monday, 8/1, the first thing I'll do is work on this merge! I hope that's okay, and again, thank you so much for all the effort on your part!

Zach10a commented 2 years ago

Yo yo!

Absolutely no worries, we all got lives to lead! Hope it's all going well, the weather over here is far too hot to work on anything anyway so you're probably doing me a favour!

Enjoy your last two weeks!

camUrban commented 2 years ago

Hey @Zach10a, I hope you survived the hot weather unscathed! I'm now finished up with work and am ready to start helping with this! Here are my initial thoughts:

camUrban commented 2 years ago

Also, would you mind changing the filenames for mainWindow.py and mainWindow.ui to main_window.py and main_window.ui? Unless there's a reason not to, I think we should try to stick with lowercase with underscores file naming standard. What are your thoughts?

Zach10a commented 2 years ago

Hi Cam,

Just out of contact until the 12th Aug, will have a look at these changes when I am back.

Glad you're well

Zach10a commented 2 years ago

Hi Cam, had an hour out so have had a look.

All of those should be actioned in the next few commits. I'll be in and out of comms for the next week or so, will get back to you on any comments as soon as I can!

Zach10a commented 1 year ago

@camUrban Ok so I've gone a little further than I should have on this branch, but it saves you having to look at more PRs. We now have a fully-fledged installer and build instructions that can install the program to the user's computer with no use of the command line. It can show all examples through the help folder. If you're happy with this being V1.0.0 (that is what I've designated it atm) then the installer .exe can be appended to the release section on the front page alongside the source code.

Looking forward to moving on with other aspects of the project!

Zach10a commented 1 year ago

@camUrban any movement on this?

Zach10a commented 1 year ago

Hey @camUrban, would there be any way to resolve this?

camUrban commented 1 year ago

Hi @Zach10a. So sorry about the long wait on this. I've been in my first semester of grad school and have been struggling to find the time to work on this. This is my last week of classes. I'll take another look on Monday, 12/12, which is my first day of winter break. Thanks for all the work and your patience.

Zach10a commented 1 year ago

No worries dude! Hope you're enjoying, have a good Christmas period.

camUrban commented 1 year ago

I just merged the pull request. Great job, and thank you again! I will close this thread, but I may open a new issue in the next few days as I start prepping a new major version release with your contributions.