Skip to content

minimize redundant autosaves#78

Open
AdrianGroty wants to merge 1 commit into
devottys:masterfrom
AdrianGroty:minimal-save
Open

minimize redundant autosaves#78
AdrianGroty wants to merge 1 commit into
devottys:masterfrom
AdrianGroty:minimal-save

Conversation

@AdrianGroty
Copy link
Copy Markdown
Contributor

@AdrianGroty AdrianGroty commented May 24, 2026

  • DrawingSheet.init: adds self.autosaved=True (line 97-99)
  • DrawingSheet.setModified: flips autosaved=False (line 101-103)
  • Drawing.autosave: skips save when source.autosaved already True; sets True after save (line 319-324)
  • Drawing.reload: clears source.autosaved=True post-load to discard load-induced dirty flag (line 454)
  • last_autosave init: int → time.time so open-time anchors interval check (line 835)

+claude

Current behavior is for drawings to autosave immediately on opening and again at every autosave interval, whether or not edits have been made. With a short interval and/or a session left open for a long time, the autosave dir can quickly become unwieldy, and any specific point in the history can be a needle in a haystack. These changes prevent autosave from firing until an edit has been made AND at least one autosave interval has elapsed since load or the last autosave.

  - DrawingSheet.__init__: adds self.autosaved=True (line 97-99)
  - DrawingSheet.setModified: flips autosaved=False (line 101-103)
  - Drawing.autosave: skips save when source.autosaved already True; sets True after save (line 319-324)
  - Drawing.reload: clears source.autosaved=True post-load to discard load-induced dirty flag (line 454)
  - last_autosave init: int → time.time so open-time anchors interval check (line 835)

+claude
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.

1 participant