forked from MTG/sms-tools
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[MTG#36] apply same fix for transformations_GUI.py
- Loading branch information
Showing
1 changed file
with
15 additions
and
10 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
aa10565
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think you'll want a similar change for models_GUI.py
aa10565
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @stephenhandley I've done that on an earlier commit 0775a7b. I haven't made a PR of my osx branch yet because I'm still working through the course.
Hmm, you made me look at this commit again. I was trying to avoid white-space patches (at least mixed in with other changes), but they obviously snuck in here!
aa10565
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tardate ah, should have looked more closely at the other commits, glad it ended up being helpful tho
aa10565
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
np, thanks again. btw I've reverted the whitespace on a new branch https://github.com/tardate/sms-tools/tree/osxcompat and I'm going to delete this branch.
NB: that means this commit will disappear, and I suspect this conversation will get "lost" also.