Skip to content

Conversation

@taiya
Copy link
Collaborator

@taiya taiya commented Mar 4, 2025

Few "subscription based" goodies that end up eventually in every paper repo.
Will add more as consolidate between ~10 ICCV submissions

@taiya taiya changed the title perks Fleshed out preamble.tex Mar 4, 2025
preamble.tex Outdated
% \definecolor{purple}{rgb}{0.65,0,0.65}
% \definecolor{dark_green}{rgb}{0, 0.5, 0}
% \definecolor{orange}{rgb}{0.8, 0.6, 0.2}
% \definecolor{red}{rgb}{0.8, 0.2, 0.2}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why override pre-defined colours?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

where are they pre-defined? These don't cause conflicts in compile

Copy link
Collaborator

Choose a reason for hiding this comment

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

The "xcolor" package predefines some basic colors, including red, purple, orange, pink, and teal. We should override these defaults.

image

Source: Named colors provided by the xcolor package

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yah I think you are right, this was left for legacy reason, but indeed, better to leave a pointer where to get pre-defined colors

Copy link
Collaborator

@cr333 cr333 left a comment

Choose a reason for hiding this comment

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

It might be good to add a comment at the top of preamble.tex explaining what the file is for and that it is okay to uncomment anything if so desired.

% \DeclareMathOperator*{\argmax}{arg\,max}

%%
%% Tigthen underline
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
%% Tigthen underline
%% Tighten underline

% --- inline annotations
%
%%
%% fine-tune paragraph spacing
Copy link
Collaborator

Choose a reason for hiding this comment

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

Capitalise for consistency

Suggested change
%% fine-tune paragraph spacing
%% Fine-tune paragraph spacing

% \renewcommand{\paragraph}[1]{\vspace{.5em}\noindent\textbf{#1.}}

%%
%% globally adjusts space between figure and caption
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
%% globally adjusts space between figure and caption
%% Globally adjusts space between figure and caption

@taiya taiya merged commit 4abe0a2 into main May 7, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants