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

Double ETA in predicted time #42

Open
WimVDK opened this issue Apr 29, 2022 · 3 comments · May be fixed by #51
Open

Double ETA in predicted time #42

WimVDK opened this issue Apr 29, 2022 · 3 comments · May be fixed by #51
Labels
bug Something isn't working

Comments

@WimVDK
Copy link

WimVDK commented Apr 29, 2022

Describe the bug
afbeelding

Should be pretty self-explanatory. After that newest update

To Reproduce
Steps to reproduce the behavior:

  1. use v2.2.0
  2. start any print.
  3. See the double ETA

Expected behavior
no double ETA of course

Screenshots
See above

Environment information:

  • Octoprint Version: 1.7.3
  • Browser: Not relevant but 99.0.1
  • Python Version: [3.7.3

Additional context
Add any other context about the problem here.

@WimVDK WimVDK added the bug Something isn't working label Apr 29, 2022
@ramosglauco
Copy link

Hi. First congratulations for the plugin, it's really useful! I have the same problem described above. Could you remove the "ETA is" in the next update? Thanks.

@leoheck
Copy link

leoheck commented Mar 2, 2023

Hi guys, I just saw this right now. Do you know any workaround to remove this duplicated item ETA string?

image

@rewolff
Copy link

rewolff commented May 7, 2024

Technically I suspect that the first ETA: is "the plugin called ETA is saying the following" and that this plugin then produces the string: "ETA is ...." .

That said.... "ETA: 10:05:23 tomorrow" is exactly what a human would want to see. so just removing "ETA is " from the output would simply suffice.

@rewolff rewolff linked a pull request May 7, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants