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

Caching 2020 chapter. #1609

Merged
merged 118 commits into from
Dec 8, 2020
Merged

Caching 2020 chapter. #1609

merged 118 commits into from
Dec 8, 2020

Conversation

raghuramakrishnan71
Copy link
Contributor

@raghuramakrishnan71 raghuramakrishnan71 commented Dec 2, 2020

Closes #917

@raghuramakrishnan71 raghuramakrishnan71 self-assigned this Dec 2, 2020
@raghuramakrishnan71 raghuramakrishnan71 added ASAP This issue is blocking progress generate chapters Used to group PRs just used to generate chapters labels Dec 2, 2020
@tunetheweb tunetheweb added writing Related to wording and content and removed generate chapters Used to group PRs just used to generate chapters labels Dec 2, 2020
@tunetheweb tunetheweb added this to the 2020 Content Writing milestone Dec 2, 2020
@tunetheweb tunetheweb changed the title Caching markdown. Caching 2020 chapter. Dec 2, 2020
@tunetheweb
Copy link
Member

Is this still a work in progress @raghuramakrishnan71 ? Looks like it only has the first few sections. Can oyu change this PR to Draft until it's ready for review?

@rviscomi rviscomi marked this pull request as draft December 2, 2020 16:32
@raghuramakrishnan71
Copy link
Contributor Author

raghuramakrishnan71 commented Dec 3, 2020

@rviscomi @bazzadp @roryhewitt - sending an update.

  • markdown for 100% content done.
  • closed open review comments from reviewers.

@csswizardry @jzyang @jaisanth @Soham-S-Sarkar have completed draft markdown, and done the final edits for closing the review comments. All the images are also added to the repository.

Released for editing.

@rviscomi
Copy link
Member

rviscomi commented Dec 7, 2020

Fig 20.20 and 20.21 are appearing truncated

@raghuramakrishnan71 this sounds like the charts may have been created with wider dimensions than the default (600x371). If the charts are non-standard then the figure syntax should explicitly set the height/width arguments. See https://github.com/HTTPArchive/almanac.httparchive.org/wiki/Figures-Guide#charts-images-and-callout-figures-examples for more info.

@tunetheweb
Copy link
Member

tunetheweb commented Dec 7, 2020

@bazzadp Fig 20.20 and 20.21 are appearing truncated (in the right) here: https://20201207t175252-dot-webalmanac.uk.r.appspot.com/en/2020/caching. I rechecked, the saved images are complete in the static folder also.

  • 14_resource_age_party_and_type_wise_groups_1st_party.png
  • 15_resource_age_party_and_type_wise_groups_3rd_party.png

@raghuramakrishnan71

Fig 20.20 and 20.21 are appearing truncated

@raghuramakrishnan71 this sounds like the charts may have been created with wider dimensions than the default (600x371). If the charts are non-standard then the figure syntax should explicitly set the height/width arguments. See https://github.com/HTTPArchive/almanac.httparchive.org/wiki/Figures-Guide#charts-images-and-callout-figures-examples for more info.

I've changed them to 600 by 371 again in the sheets. No reason for them not to be that width. I've also changed the Title of the chart to be sentence case (1st letter capitalised but test lower case).
Ok.

You'll need to retake the fallback images. Note I moved the images to standard name format so this was resource-age-party-and-type-wise-groups-1st-party.png and resource-age-party-and-type-wise-groups-3rd-party.png
Ok, Fallback images retaken for both and added.

@tunetheweb
Copy link
Member

@raghuramakrishnan71 I think you might have missed some more of my feedback:

image

And pro-tip, you can batch accept all these in the Files tab, and then submit in one go – which is a lot quicker than committing them one by one.

@rviscomi rviscomi marked this pull request as ready for review December 8, 2020 00:05
raghuramakrishnan71 and others added 3 commits December 8, 2020 09:05
and special case for header keywords.

Co-authored-by: Barry Pollard <[email protected]>
Co-authored-by: Barry Pollard <[email protected]>
@raghuramakrishnan71
Copy link
Contributor Author

@bazzadp

  • Have closed all comments and rechecked (except two, waiting on @roryhewitt for that). let me know if any item still appears pending.
  • Thanks for the batch commit tip
  • Looks like i used the review option unknowingly. Apologies.
    image

@tunetheweb
Copy link
Member

Mergeing this. We can make further edits in Copy editing.

@tunetheweb tunetheweb merged commit d77e6b1 into main Dec 8, 2020
@tunetheweb tunetheweb deleted the caching-2020-markdown branch December 8, 2020 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ASAP This issue is blocking progress writing Related to wording and content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Caching 2020
4 participants