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

Development roadmap/plan for this plugin #1

Closed
13 tasks done
cp2004 opened this issue Sep 23, 2020 · 4 comments
Closed
13 tasks done

Development roadmap/plan for this plugin #1

cp2004 opened this issue Sep 23, 2020 · 4 comments

Comments

@cp2004
Copy link
Owner

cp2004 commented Sep 23, 2020

I will be documenting my initial plans for development of this plugin in this ticket - please don't open issues for features that I already have on this list!

  • Python 3 compatibility
  • Fix some UI issues
  • Move tab from settings to its own tab
  • Add new maintainer dialog
  • Work out observed performance issues:
    • Issue commands on tab click (not app startup) to speed up load times Not required - re-written version will not need to do this at all. Requests are async, and parsing is done server side.
    • Send data to server, for persistence - means reloading does not have to parse all the data again, sending M5xx commands
  • Utilise server side
    • Allows for a more powerful implementation, where data is persisted across reloads.
    • Should allow you to view settings while the printer is printing.
    • Should solve performance issues
  • UI modifications, to make it more streamlined & easier to use
    • Display more helpful error messages (Not Marlin/No EEPROM/Error reading data etc.)
    • Make settings view-only during printing - requires python server side storage of data.
  • Test compatibility with latest Marlin
  • Sort out version number detection - get numbers for major/minor/patch etc. (remember Marlin 2.x used 4 sections - e.g. 2.0.5.3) so that there is less chance of error
    In the re-write of this plugin, there is no longer the need to detect versions.
@dchauran
Copy link

dchauran commented Oct 9, 2020

"Test compatibility with latest Marlin"

For me at least, this plugin is completely nonfunctional running marlin from bugfix-2.0.x as of last night, and was nonfunctional when loaded from the old repo as well.

When I change a setting and click upload, it runs m500 but I don't see it actually issuing any command to change the settings I changed. When I click backup, the file only contains:

Send: M504

Recv: echo:EEPROM OK
Recv: ok,Send: M501

I can probably fix these issues, but wanted to make sure you aren't actively looking at them before I dive in.

@cp2004 cp2004 changed the title [Meta] Development roadmap/plan for this plugin Development roadmap/plan for this plugin Oct 12, 2020
@cp2004
Copy link
Owner Author

cp2004 commented Oct 12, 2020

Hey @dchauran - the backup problem is tracked in #8, I haven't had a chance to dig deep since I am working through implementing the EEPROM stuff in the virtual printer, so I don't have to develop on my production machine for this one. Its almost there, just needs all of the commands now. I'm not actively looking into them right now so if you wanted to take a look, by all means go ahead.

@dchauran
Copy link

dchauran commented Oct 12, 2020 via email

@cp2004
Copy link
Owner Author

cp2004 commented Dec 13, 2020

Everything on this roadmap is covered for now, as of the release of 3.0.0

Closing, details at #15

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

No branches or pull requests

2 participants