--- Begin Message ---
|
Subject: |
12.2.4; Wrong DPI calculation for mixed-DPI multi-monitor setups |
|
Date: |
Fri, 01 Jan 2021 14:42:36 +0100 |
|
User-agent: |
mu4e 1.4.10; emacs 28.0.50 |
preview-latex calculates DPI in `preview-get-geometry` by calling
`display-pixel-width`, `display-mm-width` and analogous height-related
functions. According to documentation:
For graphical terminals, note that on "multi-monitor" setups this
refers to the pixel width for all physical monitors associated
with DISPLAY. To get information for each physical monitor, use
display-monitor-attributes-list.
This results in incorrent DPI values for multi-monitor setups with pure GTK
fork of Emacs. Please see this comment
(https://github.com/masm11/emacs/issues/90#issuecomment-753307864) on GitHub
for specifics. As a suggested fix, I'm attaching below a version of
`preview-get-geometry` that returns DPI for the monitor holding the active
frame. This way of calculating DPI works correctly for current standard Emacs
builds and for the pure GTK version of Emacs.
(defun preview-get-dpi ()
(let* ((monitor-attrs (frame-monitor-attributes))
(mm-dims (cdr (assoc 'mm-size monitor-attrs)))
(mm-width (nth 0 mm-dims))
(mm-height (nth 1 mm-dims))
(pixel-dims (cdddr (assoc 'geometry monitor-attrs)))
(pixel-width (nth 0 pixel-dims))
(pixel-height (nth 1 pixel-dims)))
(cons (/ (* 25.4 pixel-width) mm-width)
(/ (* 25.4 pixel-height) mm-height))))
(defun preview-get-geometry ()
"Transfer display geometry parameters from current display.
Returns list of scale, resolution and colors. Calculation
is done in current buffer."
(condition-case err
(let* ((geometry
(list (preview-hook-enquiry preview-scale-function)
(preview-get-dpi)
(preview-get-colors)))
(preview-min-spec
(* (cdr (nth 1 geometry))
(/
(preview-inherited-face-attribute
'preview-reference-face :height 'default)
720.0))))
(setq preview-icon (preview-make-image 'preview-icon-specs)
preview-error-icon (preview-make-image
'preview-error-icon-specs)
preview-nonready-icon (preview-make-image
'preview-nonready-icon-specs))
geometry)
(error (error "Display geometry unavailable: %s"
(error-message-string err)))))
Emacs : GNU Emacs 28.0.50 (build 1, x86_64-redhat-linux-gnu, GTK+ Version
3.24.24, cairo version 1.16.0)
of 2020-12-30
Package: 12.2.4
current state:
==============
(setq
AUCTeX-version "12.2.4"
LaTeX-command-style '(("" "%(PDF)%(latex) %(file-line-error) %(extraopts)
%S%(PDFout)"))
image-types '(svg png gif tiff jpeg xpm xbm pbm)
preview-image-type 'png
preview-image-creators '((dvipng (open preview-gs-open
preview-dvipng-process-setup)
(place preview-gs-place) (close
preview-dvipng-close))
(png (open preview-gs-open) (place preview-gs-place)
(close preview-gs-close))
(jpeg (open preview-gs-open) (place preview-gs-place)
(close preview-gs-close))
(pnm (open preview-gs-open) (place preview-gs-place)
(close preview-gs-close))
(tiff (open preview-gs-open) (place preview-gs-place)
(close preview-gs-close))
)
preview-dvipng-image-type 'png
preview-dvipng-command "dvipng -picky -noghostscript %d -o %m/prev%%03d.png"
preview-pdf2dsc-command "pdf2dsc %s.pdf %m/preview.dsc"
preview-gs-command "/usr/bin/rungs"
preview-gs-options '("-q" "-dDELAYSAFER" "-dNOPAUSE" "-DNOPLATFONTS"
"-dPrinted"
"-dTextAlphaBits=4" "-dGraphicsAlphaBits=4")
preview-gs-image-type-alist '((png png "-sDEVICE=png16m") (dvipng png
"-sDEVICE=png16m")
(jpeg jpeg "-sDEVICE=jpeg") (pnm pbm
"-sDEVICE=pnmraw")
(tiff tiff "-sDEVICE=tiff12nc"))
preview-fast-conversion t
preview-prefer-TeX-bb nil
preview-dvips-command "dvips -Pwww -i -E %d -o %m/preview.000"
preview-fast-dvips-command "dvips -Pwww %d -o %m/preview.ps"
preview-scale-function 'texfrag-scale-from-face
preview-LaTeX-command '("%`%l \"\\nonstopmode\\nofiles\\PassOptionsToPackage{"
("," . preview-required-option-list)
"}{preview}\\AtBeginDocument{\\ifx\\ifPreview\\undefined"
preview-default-preamble "\\fi}\"%' \"\\detokenize{\"
%t \"}\"")
preview-required-option-list '("active" "tightpage" "auctex"
(preview-preserve-counters "counters"))
preview-preserve-counters nil
preview-default-option-list '("displaymath" "floats" "graphics" "textmath"
"sections"
"footnotes")
preview-default-preamble '("\\RequirePackage[" ("," .
preview-default-option-list)
"]{preview}[2004/11/05]")
preview-LaTeX-command-replacements nil
preview-dump-replacements '(preview-LaTeX-command-replacements
("\\`\\([^ ]+\\)\\(?: +\\(?:\\(--?kanji[= ][^
]+\\)\\|-\\(?:[^ \\\"]\\|\\\\.\\|\"[^\"]*\"\\)*\\)\\)*\\(.*\\)\\'" "\\1 -ini
\\2 -interaction=nonstopmode \"&\\1\" " preview-format-name ".ini \\3")
)
preview-undump-replacements '(("\\`\\([^ ]+\\)\\(?: +\\(?:\\(--?kanji[= ][^
]+\\)\\|-\\(?:[^ \\\"]\\|\\\\.\\|\"[^\"]*\"\\)*\\)\\)*.* \"\\\\input\"
\"\\\\detokenize{\" \\(.*\\) \"}\"\\'" "\\1 \\2 -interaction=nonstopmode
-file-line-error " preview-format-name " \"/AUCTEXINPUT{\" \\3 \"}\"")
)
preview-auto-cache-preamble 'ask
preview-TeX-style-dir
"/home/grz/.emacs.d/.local/straight/build-28.0.50/auctex/latex"
)
Output from running `/usr/bin/rungs -h':
GPL Ghostscript 9.53.3 (2020-10-01)
Copyright (C) 2020 Artifex Software, Inc. All rights reserved.
Usage: gs [switches] [file1.ps file2.ps ...]
Most frequently used switches: (you can use # in place of =)
-dNOPAUSE no pause after page | -q `quiet', fewer messages
-g<width>x<height> page size in pixels | -r<res> pixels/inch resolution
-sDEVICE=<devname> select device | -dBATCH exit after last file
-sOutputFile=<file> select output file: - for stdout, |command for pipe,
embed %d or %ld for page #
Input formats: PostScript PostScriptLevel1 PostScriptLevel2 PostScriptLevel3 PDF
Default output device: bbox
Available devices:
alc1900 alc2000 alc4000 alc4100 alc8500 alc8600 alc9100 ap3250 appledmp
atx23 atx24 atx38 bbox bit bitcmyk bitrgb bitrgbtags bj10e bj10v bj10vh
bj200 bjc600 bjc800 bjc880j bjccmyk bjccolor bjcgray bjcmono bmp16 bmp16m
bmp256 bmp32b bmpgray bmpmono bmpsep1 bmpsep8 ccr cdeskjet cdj1600 cdj500
cdj550 cdj670 cdj850 cdj880 cdj890 cdj970 cdjcolor cdjmono cdnj500 cfax
chp2200 cif cljet5 cljet5c cljet5pr coslw2p coslwxl cups declj250 deskjet
devicen dfaxhigh dfaxlow display dj505j djet500 djet500c dl2100 dnj650c
epl2050 epl2050p epl2120 epl2500 epl2750 epl5800 epl5900 epl6100 epl6200
eplcolor eplmono eps2write eps9high eps9mid epson epsonc escp escpage
faxg3 faxg32d faxg4 fmlbp fmpr fpng fs600 gdi hl1240 hl1250 hl7x0
hpdj1120c hpdj310 hpdj320 hpdj340 hpdj400 hpdj500 hpdj500c hpdj510
hpdj520 hpdj540 hpdj550c hpdj560c hpdj600 hpdj660c hpdj670c hpdj680c
hpdj690c hpdj850c hpdj855c hpdj870c hpdj890c hpdjplus hpdjportable ibmpro
ijs imagen inferno ink_cov inkcov itk24i itk38 iwhi iwlo iwlq jetp3852
jj100 jpeg jpegcmyk jpeggray la50 la70 la75 la75plus laserjet lbp310
lbp320 lbp8 lex2050 lex3200 lex5700 lex7000 lips2p lips3 lips4 lips4v
lj250 lj3100sw lj4dith lj4dithp lj5gray lj5mono ljet2p ljet3 ljet3d ljet4
ljet4d ljet4pjl ljetplus ln03 lp1800 lp1900 lp2000 lp2200 lp2400 lp2500
lp2563 lp3000c lp7500 lp7700 lp7900 lp8000 lp8000c lp8100 lp8200c lp8300c
lp8300f lp8400f lp8500c lp8600 lp8600f lp8700 lp8800c lp8900 lp9000b
lp9000c lp9100 lp9200b lp9200c lp9300 lp9400 lp9500c lp9600 lp9600s
lp9800c lps4500 lps6500 lq850 lxm3200 lxm5700m m8510 md1xMono md2k
md50Eco md50Mono md5k mgr4 mgr8 mgrgray2 mgrgray4 mgrgray8 mgrmono miff24
mj500c mj6000c mj700v2c mj8000c ml600 necp6 npdl nullpage oce9050 oki182
oki4w okiibm paintjet pam pamcmyk32 pamcmyk4 pbm pbmraw pcl3 pclm pcx16
pcx24b pcx256 pcxcmyk pcxgray pcxmono pdfimage24 pdfimage32 pdfimage8
pdfwrite pdfwrite pdfwrite pgm pgmraw pgnm pgnmraw photoex picty180 pj
pjetxl pjxl pjxl300 pkm pkmraw pksm pksmraw plan plan9bm planc plang
plank planm plib plibc plibg plibk plibm png16 png16m png256 png48
pngalpha pnggray pngmono pngmonod pnm pnmraw ppm ppmraw pr1000 pr1000_4
pr150 pr201 ps2write psdcmyk psdcmyk16 psdcmykog psdrgb psdrgb16
pwgraster pxlcolor pxlmono r4081 rinkj rpdl samsunggdi sj48 spotcmyk
st800 stcolor t4693d2 t4693d4 t4693d8 tek4696 tiff12nc tiff24nc tiff32nc
tiff48nc tiff64nc tiffcrle tiffg3 tiffg32d tiffg4 tiffgray tifflzw
tiffpack tiffscaled tiffscaled24 tiffscaled32 tiffscaled4 tiffscaled8
tiffsep tiffsep1 txtwrite uniprint xcf xes xpswrite
Search path:
/usr/share/ghostscript/Resource/Init : /usr/share/ghostscript/lib :
/usr/share/ghostscript/Resource/Font : /usr/share/ghostscript/fonts :
/usr/share/fonts/urw-base35 : /usr/share/fonts/google-droid-sans-fonts
Ghostscript is also using fontconfig to search for font files
For more information, see /usr/share/doc/ghostscript/Use.htm.
Please report bugs to bugs.ghostscript.com.
--- End Message ---
--- Begin Message ---
|
Subject: |
Re: bug#45596: 12.2.4; Wrong DPI calculation for mixed-DPI multi-monitor setups |
|
Date: |
Mon, 04 Jan 2021 21:27:11 +0100 |
|
User-agent: |
mu4e 1.5.7; emacs 28.0.50 |
Grzegorz Kowzan <grzegorz@kowzan.eu> writes:
Hi Grzegorz,
>> I guess the problem is that mm-size in the `frame-monitor-attributes'
>> return value differs from `display-mm-height' and `display-mm-width'.
>
> 1) Yes, `display-pixel/mm-width/height` functions are defined in terms
> of Xlib calls, whereas `frame-monitor-attributes` calls Gdk functions
> (if Emacs was compiled with Gtk support) and only as a fallback it
> calls Xlib. This is why we have this inconsistency. The standard DPI
> for Xorg server is 96 regardless of the actual value, so for a given
> resolution and assumed DPI Xorg gives fake mm width and
> height. Essentially, the current calculation of DPI in auctex looks
> pointless to me...
Ok, I see.
> With pure Gtk port we get actual physical pixel width/height and
> actual mm width/height, so I guess when auctex detects pgtk port, it
> can either ignore these values and hardcode DPI of 96 or make use of
> the actual DPI. I suppose there is something to be said for not
> changing auctex's behaviour and hardcoding the value, but I would
> prefer to use the actual DPI at least as an option (see below).
That's what I did. Essentially I've applied your patch with an
additional fboundp check for `frame-monitor-attributes'. If it's
available (24.4), then your variant is used, if not, the previous
variant basically always saying 96 DPI is used.
So let's see if we get any complaints with the next ELPA release.
I'll close this report now. Thanks again for the contribution!
Bye,
Tassilo
--- End Message ---