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 "PDF Report" saves PNG image #1718

Closed
1 of 4 tasks
l2jc opened this issue Oct 16, 2024 · 11 comments
Closed
1 of 4 tasks

Autosave "PDF Report" saves PNG image #1718

l2jc opened this issue Oct 16, 2024 · 11 comments
Labels
Milestone

Comments

@l2jc
Copy link

l2jc commented Oct 16, 2024

Problem Description

Using autosave to save roast logs and PDF Reports results in the logs being saved but the PDF Report being a PNG image of the the roast chart. Using just "PDF" saves a PDF for the image but "PDF Report" which contains more information of the roast log doesn't appear to work.

Steps To Reproduce

No response

Artisan Version

3.0.0

Computer OS and Version

Windows 11

Connected devices or roasting machine

Mill City Roaster 500g

Files

  • A sample profile *.alog file, renamed to *.alog.txt is attached.

  • A settings file (Help>> Save Settings) *.aset, renamed to *.aset.txt is attached.

  • Screenshots are attached.

  • No files are attached.

@roasterdave
Copy link
Contributor

I can not reproduce this here on Windows 11 using Artisan v3.0.0 or later.

Please provide a sample profile .alog file where this happened and your .aset settings file (Help>> Save Settings). Rename each file to end in ".txt" then drag-drop the files onto a reply box in this thread here on GitHub. Do not send in an email reply.

@beanoccio
Copy link

I have been using all versions of artisan V3.0.0 up to the current continuous build Artisan 3.0.3 (899462a) with autosave for the last 6 weeks. (Windows 11)

I checked the folder with the autosaved PDFs. There are only PDF files there. Maybe a reset to factory settings and reloading the settings file will help.

@l2jc
Copy link
Author

l2jc commented Oct 17, 2024

Sample alog and aset files attached.

I've just tried a factory reset and reloading settings and it still produces the same issue. To clarify, autosaving "PDF" works fine. It is when autosaving "PDF Reports" that it saves a PNG of the roast chart...no report file to be found.

1760 Ethiopia Yirgacheffe Halo Beriti 24-10-16.txt
artisan-settings 20241017.txt

@beanoccio
Copy link

Ok, autosaving "PDF Reports" was not used in my earlier Roasts with v3.0.0. (only autosaving "PDF")
I have now tested autosaving "PDF Reports" in the current Artisan 3.0.3 (899462a). Everything seems to be working fine (again).
A PDF with all the details is saved. Identical to how it works manually via the menu.

@roasterdave
Copy link
Contributor

roasterdave commented Oct 17, 2024

@l2jc Thank you for the additional information. I still can not reproduce this with Artisan 3.0.0 and did confirm we are both running the same build.

Inspecting the code there exists a check for a necessary library needed specifically to write the PDF Roast Report. If the library is missing the PNG file will be written. I have verified that the Artisan v3.0.0 release installs this library and is thus able to write the PDF Roast report. Your installation may be corrupt somehow. I recommend reinstalling Artisan.

Before reinstalling, it will help me to try to diagnose what happened to your installation if you would do the following.

  • Open a command window (click the Windows key and type cmd followed by Enter)
  • At the prompt enter: dir /s "\program files\artisan\_internal" > dirlist.txt
  • Optionally zip that file and then drag it onto a reply box in this issue.
  • You should then try reinstalling Artisan and check if the problem is resolved.

Thanks for your support.

@Terracotta-6
Copy link

Terracotta-6 commented Oct 18, 2024

Artisan v3.0.3 (56b381d, 0) During the function test, we inspected multiple output formats. Autosave performs normally in PDF, PDF Report, SVG, PNG, JPEG, CSV and JSON formats and can be output smoothly. However, it is currently found in the test cannot output in BMP format. In addition, file list files of the Win10 system.
dirlist.txt

@MAKOMO
Copy link
Member

MAKOMO commented Oct 18, 2024

Support for the image format BMP has been removed

@l2jc
Copy link
Author

l2jc commented Oct 21, 2024

@roasterdave Here is the dirlist.

dirlist.txt

@roasterdave
Copy link
Contributor

Thank you for providing this additional information. There are no files missing from the installation. That makes this all the more puzzling. If you look at the menu File>> Report>> Roast is the PDF choice active or is it grayed out?

At this point I recommend installing the continuous build. If the problem persists we will keep working on it.

@l2jc
Copy link
Author

l2jc commented Oct 22, 2024

The File > Report > Roast > PDF is available. That is what I've been using (manually) this whole time. It was only recently that I saw there was an autosave function for it too that I wanted to try it.

@MAKOMO MAKOMO added the bug label Oct 23, 2024
@MAKOMO MAKOMO added this to the v3.0.4 milestone Oct 23, 2024
MAKOMO added a commit that referenced this issue Oct 23, 2024
…Report if "PDF Report" was selected as type under "Save also" in the autosave dialog (Issue #1718)

- adds BLE read and removes the blacklisting
- splits devices into ColorTrack Serial and ColorTrack BLE promoted as main devices types
- adds ColorTrack BT laser data mapping and temp/hum data logging
- RS time starts always on PID ON
- fixes pidModeInit (Discussion #1721)
- updates pyright, mypy, pymodbus
@MAKOMO
Copy link
Member

MAKOMO commented Oct 23, 2024

We finally could reproduce the issue (it happens not on autosave at the end of the roast, but on pressing Save from the File menu manually) and fix it. The fix is contained in the just being produced Continuous Build.

Thanks for reporting!

@MAKOMO MAKOMO closed this as completed Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants