Skip to content
This repository has been archived by the owner on Feb 2, 2022. It is now read-only.

Fix a variety of wx related errors #143

Closed
wants to merge 10 commits into from

Conversation

jmartens
Copy link

@jmartens jmartens commented Nov 2, 2021

While running dicompyler from source with the requirements from requirements.txt installed I get a lot of errors, this PR should address them to make dicompyler stop throwing wx related errors.

Among others it fixes:

  • wx.lib.pubsub being deprecated in favor of PyPubSub
  • fix layout errors for options not valid in horizontal box sizer
  • use wx.Cursor in favor of deprecated wx.CursorFromImage()
  • remove wxALIGN* where WX_EXPAND is also set

This also incorporates the fixes of #131.

@bastula
Copy link
Owner

bastula commented Nov 4, 2021

Thanks for working on this. I'll take a look shortly and test out the changes.

@jmartens jmartens closed this Aug 30, 2023
@jmartens jmartens deleted the fixes/wx-errors branch August 30, 2023 06:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants