wacv-pcs / WACV-2023-Author-Kit

16 stars 7 forks source link

Line number misalignment #1

Open WesleyHsieh0806 opened 2 years ago

WesleyHsieh0806 commented 2 years ago

Hi, I used the overleaf template provided in this repo and compiled the project. However, the line number seemed to be too close to the text (see the figure below). Is the format correct, or did I miss anything? image

xuyu0010 commented 2 years ago

Same problem here, the pdf produced by compiling the given tex file seems to have this problem. The sampled pdf does not have this problem.

craymichael commented 2 years ago

To resolve this, change your TeX Live Version to 2020. In Overleaf, you can do this by:

  1. Click Menu in the upper-left corner
  2. Find the Settings submenu and click the dropdown
  3. Select 2020
  4. Recompile

Unsure what needs to be done to work with 2021, but this is the quick fix.

Edit: If you then get the error Error: Command \@EveryShipout@Hook already defined see the solution here.

zhangliyun9120 commented 2 years ago

To resolve this, change your TeX Live Version to 2020. In Overleaf, you can do this by:

  1. Click Menu in the upper-left corner
  2. Find the Settings submenu and click the dropdown
  3. Select 2020
  4. Recompile

Unsure what needs to be done to work with 2021, but this is the quick fix.

Edit: If you then get the error Error: Command \@EveryShipout@Hook already defined see the solution here.

But changed TeX Live Version to 2020, and it seems to be still not good:

test

Is there any other better solution?thx

trougnouf commented 2 years ago

To resolve this, change your TeX Live Version to 2020. In Overleaf, you can do this by:

1. Click Menu in the upper-left corner

2. Find the **Settings** submenu and click the dropdown

3. Select 2020

4. Recompile

Unsure what needs to be done to work with 2021, but this is the quick fix.

Edit: If you then get the error Error: Command \@EveryShipout@Hook already defined see the solution here.

Same issue. I don't think this workaround can be applied to a local installation (= texlive 2022) so a better solution is needed.

xingyuuchen commented 2 years ago

Just modify the file wacv.sty line 221 from

% \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+30pt},\LenToUnit{-\wacvruleroffset}){%right ruler.

zhangliyun9120 commented 2 years ago

Just modify the file wacv.sty line 221 from

% \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+30pt},\LenToUnit{-\wacvruleroffset}){%right ruler.

The final solution should be:

1, Still setting TeX Live Version to 2021

2, % \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+20pt},\LenToUnit{-\wacvruleroffset}){%right ruler

xingyuuchen commented 2 years ago

Just modify the file wacv.sty line 221 from % \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+30pt},\LenToUnit{-\wacvruleroffset}){%right ruler.

The final solution should be:

1, Still setting TeX Live Version to 2021

2, % \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+20pt},\LenToUnit{-\wacvruleroffset}){%right ruler

Ah..., idk why you changed 30 to 20, 30 is from WACV's official style. If you check other CVF conference latex template e.g. CVPR you could also will find it 30.

trougnouf commented 2 years ago

Just modify the file wacv.sty line 221 from

% \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+30pt},\LenToUnit{-\wacvruleroffset}){%right ruler.

That works, thank you!

zhangliyun9120 commented 2 years ago

Just modify the file wacv.sty line 221 from % \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+30pt},\LenToUnit{-\wacvruleroffset}){%right ruler.

The final solution should be: 1, Still setting TeX Live Version to 2021 2, % \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+20pt},\LenToUnit{-\wacvruleroffset}){%right ruler

Ah..., idk why you changed 30 to 20, 30 is from WACV's official style. If you check other CVF conference latex template e.g. CVPR you could also will find it 30.

Because when setting 30 the left and right margin is not the same, setting 20 will be same, as the follow comparison:

Setting 30: 1

Setting 20: 2

xingyuuchen commented 2 years ago

Just modify the file wacv.sty line 221 from % \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+30pt},\LenToUnit{-\wacvruleroffset}){%right ruler.

The final solution should be: 1, Still setting TeX Live Version to 2021 2, % \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+20pt},\LenToUnit{-\wacvruleroffset}){%right ruler

Ah..., idk why you changed 30 to 20, 30 is from WACV's official style. If you check other CVF conference latex template e.g. CVPR you could also will find it 30.

Because when setting 30 the left and right margin is not the same, setting 20 will be same, as the follow comparison:

Setting 30: 1

Setting 20: 2

You'd better report a bug to CVPR, ICCV, ECCV, WACV, because 30 is what thousands of previous papers used.

zhangliyun9120 commented 2 years ago

Just modify the file wacv.sty line 221 from % \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+30pt},\LenToUnit{-\wacvruleroffset}){%right ruler.

The final solution should be: 1, Still setting TeX Live Version to 2021 2, % \put(\LenToUnit{\textwidth\kern 30pt},\LenToUnit{-\wacvruleroffset}){%right ruler to \put(\LenToUnit{\dimexpr \textwidth+20pt},\LenToUnit{-\wacvruleroffset}){%right ruler

Ah..., idk why you changed 30 to 20, 30 is from WACV's official style. If you check other CVF conference latex template e.g. CVPR you could also will find it 30.

Because when setting 30 the left and right margin is not the same, setting 20 will be same, as the follow comparison: Setting 30: 1 Setting 20: 2

You'd better report a bug to CVPR, ICCV, ECCV, WACV, because 30 is what thousands of previous papers used.

Reported, and got the response as follows:

Do you mean that the line numbers on the right side are slightly further away from the text than the line numbers on the left side? If so, this is not really a bug -- it's the same style that other vision conferences like CVPR have used for many years. :) The line numbers only appear in the copy for peer review, so it won't affect your final paper.

I think maybe just keeping the original 30 is ok. btw, I found the margin of ECCV is fine.