Skip to content

Commit

Permalink
Slightly more eye-friendly styling (#14)
Browse files Browse the repository at this point in the history
* Slightly more eye-friendly styling

I increased the font sizes of headers and main text,
and constrained the max width of the div to be 700px.
Makes for easier reading.

* Adjusted width of figures

* New speed comparison figure

- line plot instead of bar plot for speed comp
- bigger font size for trace plot

* removed unnecessary styling

Co-authored-by: Arkadij Kummer <43340666+ElArkk@users.noreply.github.com>
  • Loading branch information
ericmjl and ElArkk committed Feb 12, 2020
1 parent aeae676 commit d305762
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 3 deletions.
Binary file added paper/figures/rep_trace_lf.png
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added paper/figures/speed_comparison.png
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
4 changes: 2 additions & 2 deletions paper/paper.md
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ A formal speed comparison using the same CPU is available below.
in the multi-sequence case,
whereas in the original implementation the sequences were transformed
one at a time.
](./figures/speed_barplot.png)
](./figures/speed_comparison.png#center){width=50%}

We also needed to check that our reimplementation correctly embeds sequences.
To do so, we ran a dummy sequence
Expand All @@ -214,7 +214,7 @@ is a trace of 1900-long embedding.
almost perfectly, a small constant was added to the UniRep values,
such that both traces become visible. The inset shows
50 out of the total 1900 dimensions.
](./figures/rep_trace.png)
](./figures/rep_trace_lf.png)

We also verified that the embeddings of our reimplementation
were informative for top models,
Expand Down
16 changes: 15 additions & 1 deletion paper/style.css
Original file line number Diff line number Diff line change
Expand Up @@ -14,16 +14,25 @@ p,
li,
ol {
line-height: 1.5;
font-size: 1.5rem;
}

p.caption {
font-style: italic;
}

h1.title {
h1 {
font-size: 3rem;
}

h2 {
font-size: 2.5rem;
}

h3 {
font-size: 2rem;
}

pre code {
background-color: #e8e8e8;
}
Expand All @@ -34,6 +43,11 @@ img {
text-align: center;
}

img[src*="#center"] {
display: block;
margin: auto;
}

table {
width: 100%;
margin: 1.667em 0 1em;
Expand Down

0 comments on commit d305762

Please sign in to comment.