neuralbertatech / neuralbertatech/natKit

Add a better abstraction to Window

Open
#6 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Maintenance
Dominant language
Svelte
Stars
4
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Currently the Window class in src/gui/window/window.py abstracts away the process of creating a window to host graphs using PyQt5. However it does not handle the setup code needed to run it. At the moment the user of the window needs to create a QApplication and run exec() on it. This dependency should be pulled into Window

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with src/gui/window/window.py and trace how callers currently create QApplication and invoke exec() when hosting graphs. Determine the intended Window lifecycle from the existing abstraction; the work is done when callers no longer need to provide this setup and the graph-hosting window still runs correctly.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
desktop
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.