Octoprint_Pinput_Shaping a Plugin to Test Input Shaping with Marlin

[CHD]

With your permission, I will outline the deficiencies I have observed in the plugin's graphical design when working with a web browser. From an analytics standpoint, I believe the plugin works flawlessly.

Here are the issues:

On the 128x64 LCD screens (which many printers are equipped with), the process status message is not fully displayed. These screens have a horizontal limit of 21 characters.

Changing the final frequency does not affect the resulting graph. The final frequency on the graph is always 200 Hz, which is inconvenient when only the relevant portion of the data is needed.

After clicking the Sensor Values button and navigating away from the OctoPrint page and then back, the plugin displays hundreds of sensor values. It would be helpful to limit these values to about 10 lines.

[CHD]

Similarly, on the downloaded graph file, the informational text labels overlap the scale.

[CHD]

When opening the graph in zoom mode, it appears clipped or cut off. The missing section is not displayed even on a secondary monitor. Furthermore, the download button is outside the frame. The informational text labels overlap the scale.

[CHD]

The plotter icons overlap the text labels.

Sorry for having to split what I wrote - newbie limitations

Thank you once again for your assistance and the tremendous work you are doing for us.

[jneilliii]

Might make sense to post these as issues on the Plugin's GitHub Repository.

https://github.com/navaismo/Octoprint-Pinput_Shaping/issues?q=sort%3Aupdated-desc+is%3Aissue+is%3Aopen

[navaismo]

Thanks for the feedback, the Plugin still in Release Candidate stage or Beta version if you prefer, so yes there are a lot of things to fix and improve.

I'm taking note of everything what you found in your testing, thanks for reporting. Will be great as @jneilliii pointed to open issues in the repo so I can track it better. In the meantime:

@CHD

On the 128x64 LCD screens (which many printers are equipped with), the process status message is not fully displayed. These screens have a horizontal limit of 21 characters.

Good to know I'm using a different LCD, so I will check to shorten the message.

@CHD

Changing the final frequency does not affect the resulting graph. The final frequency on the graph is always 200 Hz, which is inconvenient when only the relevant portion of the data is needed.

Indeed, must fix that.

@CHD

After clicking the Sensor Values button and navigating away from the OctoPrint page and then back, the plugin displays hundreds of sensor values. It would be helpful to limit these values to about 10 lines.

Yes, initially the UI of that feature was developed fo the SPI sensor which was configured to show 10 lines, then people asked to support USB sensors and forgot to change that UI behaviour.

@CHD

Similarly, on the downloaded graph file, the informational text labels overlap the scale.

@CHD

When opening the graph in zoom mode, it appears clipped or cut off. The missing section is not displayed even on a secondary monitor. Furthermore, the download button is outside the frame. The informational text labels overlap the scale.

Noted.

@CHD

The plotter icons overlap the text labels.

Let me check if Plotly can put those somewhere else.

I've been trapped trying to fix the custom firmware of my Printer so all the above improvements will take a little bit more to be fixed, there are things that must be worked on the Shaping plugin too like giving the acceleration recommendation so yeah, I need time to fix it.

Thanks again for taking time on reporting the issues.

[jneilliii]

@navaismo

Let me check if Plotly can put those somewhere else.

I don't think the modebar can be moved, but you can hide it completely.

https://plotly.com/javascript/configuration-options/#never-display-the-modebar

Otherwise, you'll probably have to modify title layout options to move the text out of the way.

[navaismo]

Thanks for the advice.