Skip to content

Remove antiquated Qt crash prevention#2573

Merged
j9ac9k merged 1 commit intopyqtgraph:masterfrom
NeilGirdhar:master
Jan 7, 2023
Merged

Remove antiquated Qt crash prevention#2573
j9ac9k merged 1 commit intopyqtgraph:masterfrom
NeilGirdhar:master

Conversation

@NeilGirdhar
Copy link
Copy Markdown
Contributor

Fixes #2572

Other Tasks

Bump Dependency Versions

Files that need updates

Confirm the following files have been either updated or there has been a determination that no update is needed.

  • README.md
  • setup.py
  • tox.ini
  • .github/workflows/main.yml and associated requirements.txt and conda environemt.yml files
  • pyproject.toml
  • binder/requirements.txt
Pre-Release Checklist

Pre Release Checklist

  • Update version info in __init__.py
  • Update CHANGELOG primarily using contents from automated changelog generation in GitHub release page
  • Have git tag in the format of pyqtgraph-
Post-Release Checklist

Steps To Complete

  • Append .dev0 to __version__ in __init__.py
  • Announce on mail list
  • Announce on Twitter

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Jan 5, 2023

Hi @NeilGirdhar

Thanks for the PR, I always love to clean up older, likely unused code. Looking through git blame I came across this commit which is also related to TensorFlow compatibility b9c7e37

I'm good with killing off this code section... I always hated messing with garbage collection, I'll leave this PR up for a day or so to see if any other maintainer has an issue with it; if not, I'll merge.

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Jan 7, 2023

Merging, thanks for debugging this @NeilGirdhar

@j9ac9k j9ac9k merged commit c509b95 into pyqtgraph:master Jan 7, 2023
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

Successfully merging this pull request may close these issues.

Conflict with tensorflow-datasets

2 participants