GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — add `__del__` for `Frame` and `ImageWidget` · Issue #450 · fastplotlib/fastplotlib

add __del__ for Frame and ImageWidget #450

New issue
Copy link
New issue
Copy link
Closed as not planned
Closed as not planned
add __del__ for Frame and ImageWidget#450
Copy link
Assignees
kushalkolar

Description

@kushalkolar
kushalkolar
opened on Mar 24, 2024
Issue body actions

and have it basically call the existing close() methods.

Reacted by Eric Thomson

Activity

  1. kushalkolar
    self-assigned this
    on Mar 27, 2024
  2. kushalkolar commented on Apr 9, 2024

    @kushalkolar
    kushalkolar
    on Apr 9, 2024
    MemberAuthor
    More actions

    I keep forgetting that this isn't how python gc works, they must explicitly call close() because __del__ is only invoked if that is the last reference that exists. When in jupyter there will be multiple references to any Figure so an explicit close() will have to be called for cleanup.

  3. kushalkolar
    closed this as not plannedon Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

  • @kushalkolar
    kushalkolar

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    You are visitor 01402520 — GeezerVison does not store the pages you look up.
    Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.