Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AutoSave Feature #490

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vinayakjeet
Copy link

@vinayakjeet vinayakjeet commented Mar 29, 2024

Work-In-Progress: Pull Request #182

New Autosave Feature for Avogadro

This pull request introduces a new autosave feature to enhance the user experience in the Avogadro application. The key aim is to reduce the risk of data loss by automatically saving the current molecule document at a specified interval of 5 minutes, especially in case of unexpected application closures.

Implementation Details:

  • Autosave Interval Setup: Utilizes QTimer to establish an autosave routine every 5 minutes.
  • Autosave Directory Determination: Automatically determines the autosave directory based on the application's data location preferences.
  • Document Format: Saves the document in the CJSON format, ensuring a compact and efficient data structure.
  • Unsaved Changes Detection: The feature is designed to activate only when there are unsaved changes, thus optimizing resource usage and avoiding unnecessary disk writes.

Integration:

  • Updates to MainWindow Class: Incorporates autosave setup and execution logic within the MainWindow class, seamlessly integrating this feature into the application's core functionality.

This addition is expected to act as a safety net for users, securing their work without the need for manual intervention and frequent saves. The autosave feature is poised to significantly enhance productivity and data security for Avogadro users.

closes #182

Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>
@vinayakjeet
Copy link
Author

Hello maintainers
Introduced an autosave feature in Avogadro to automatically save work every 5 minutes, reducing data loss risk. Contemplating further UI enhancements to indicate autosave status and considering the implementation of a DESTRUCTOR for cleanup. Feedback on logic and suggestions for UI improvements would be appreciated

Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>
@ghutchis ghutchis mentioned this pull request Dec 8, 2024
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.

Auto save feature
1 participant