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

Update content/print/overview.md #662

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

Conversation

alikstimix
Copy link
Contributor

Simple update to test my understanding of the process.

Simple update to test my understanding of the process.
There is an outstanding issue that your target printer has to be active before you start dt.  This issue may be fixed, but until then this change describes how print works today.

Also added description of how to use the filmstrip

After selecting an image in the [lighttable view](../lighttable/_index.md) you can enter the [print settings](../module-reference/utility-modules/print/print-settings.md) module to adjust printer settings and initiate printing.
Darktable only scans for printers at startup, so your target printer must be active when darktable starts in order to provide the print module with the target printer's ICC profile.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you test it yourself?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did! Several times!!

I have a colour printer that is normally switched off as I rarely print anything. When I first looked at the Print view I couldnt load any image into centre panel, so nothing could work. After some head scratching and checking of Issues in gihub, I managed to understand why.

If I turn my printer on, and let it settle down I can then start dt and see my printer in the dropdown list. Of course I still have to know to select an image in lighttable after which I get the image in Print view, plus drag and drop from the filmstrip. And I can print out a colour image just fine.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Examining the source code shows that darktable does not scan printers at startup. Therefore, your statement is incorrect.

@TurboGit
Copy link
Member

That's strange, certainly not something I see on my side. I'm away from home at the moment and see my printer in the list and I can select it.

@alikstimix
Copy link
Contributor Author

That's strange, certainly not something I see on my side. I'm away from home at the moment and see my printer in the list and I can select it.

Its on the Issue list as #15575

@alikstimix
Copy link
Contributor Author

Actually I just tested against my wife's printer, and now it all works fine.

I'll investigate more tomorow.

@paperdigits
Copy link
Contributor

paperdigits commented Jul 18, 2024 via email

@alikstimix
Copy link
Contributor Author

OK I've pinned it down repeatably.

Start with a new instance of dt, and nothing works until you
a)select and image in lightable
b) double click on a printer in the print right panel

After that it seems everything is cached and will work as expected.

Mica - I agree the manual doesnt need to be this specific, unless this is a bug/expected behaviour that won't be changing anytime soon.

@elstoc
Copy link
Contributor

elstoc commented Jul 18, 2024

We don't document bugs in the manual, only expected behaviour. If the printer is expected to be in the list at all times, then anything else is a bug. BTW my laser printer is off at the moment and I can see it and select it in the print view.

@paperdigits
Copy link
Contributor

paperdigits commented Jul 19, 2024 via email

Updated after comments from reviewers
@alikstimix
Copy link
Contributor Author

That's a good point about translation magnifying the workload after a change to the English text.

There are lots of sections that could be improved a little, but it has to be a big improvement to justify 7 translations.


This module supports the printer's ICC profile, which is somewhat mandatory if you want to obtain a high quality print close to the image displayed on the screen.

It is important to note that ICC profiles provided by the paper and/or printer manufacturers cannot be used on GNU/Linux as they are printer-driver dependent. The darktable print module uses CUPS and there are no ready-to-use ICC profiles available for this driver.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason for removing this "important" paragraph?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well the text is pretty opaque, and its hard to see what relevance it has to the user. It seems to be describing an aspect of the workflow of the module, and is not something the user can affect.

Maybe its of use to whoever is maintaning the module, but even then it needs a rewrite.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks to be useful information for users and I'm pretty sure I understand what it's saying (you cannot use manufacturer-produced ICC profiles because they're designed for the manufacturer-produced printer drivers -- i.e. for Window$ and/or Mac) . If you don't understand it, that's not necessarily a reason to remove it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Chris, You and I may understand it, but we are not the target audience for the user manual.

Users come to the print overview because they want to understand dt printing better, or have a particular problem. Therefore the text should be accessible to an intelligent non-specialist.

If the user doesn't already know what an ICC profile is, then the text in question is unlikely to be relevant to their problems, and the warning is superfluous.

If they do know a bit about ICC profiles then, in my opinion, the text is imprecise and in some ways self contradicting.

"this module supports the printer's ICC profile . . ." probably means "this module *uses . . . " and the rest of the sentence just states the obvious.

The sentence about not using the printer ICC profile on Linux is hard to understand. I only run Linux , so where does the profile fit in to my print workflow? Also, dt does not provide any means to directly access the printer profile, so what is the text warning against? If the user really wants to try to reuse the printer ICC profile, they must already have some knowledge and expertise outside of dt in order to get hold of it. In which case they wouldnt be asking the dt manual for help.

So anyway, my feeling is that the text would be much clearer and potentially more useful if it included a description of the dt workflow for printing - explaining why multiple ICC profiles are needed and how dt interacts with printer drivers. In this case, I don't think the text belongs in a section called "overview" but potentially in a new section called Print Workflow, that can include more technical information.

Overall I am just providing feedback and suggestions, you are the gatekeeper - your call!

Copy link

This pull request has not had any activity in the past 60 days and will be closed in 365 days if not updated. Please verify it has no conflicts with the master branch and rebase if needed. Please add a comment if you need help or give permission to other people to finish your work.

@github-actions github-actions bot added the no-pr-activity No activity on this PR label Sep 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-pr-activity No activity on this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants