Skip to content

Conversation

@ryantibs
Copy link
Member

Trivial changes to the forecast demo blog post. Remove the model names from the aligned equation block, incorporate them into the paragraph below.

@ryantibs ryantibs requested a review from sgratzl December 29, 2020 02:57
@ryantibs ryantibs changed the base branch from dev to main December 29, 2020 02:58
@netlify
Copy link

netlify bot commented Dec 29, 2020

✔️ Deploy preview for cmu-delphi-main ready!

🔨 Explore the source changes: e5bf6ad

🔍 Inspect the deploy logs: https://app.netlify.com/sites/cmu-delphi-main/deploys/5feb2b871f267c0007f4d951

😎 Browse the preview: https://deploy-preview-200--cmu-delphi-main.netlify.app

@ryantibs
Copy link
Member Author

I'm not sure what you want me to merge into, dev or main. When I used dev (the default), it looks like @sgratzl there was a bunch of commits from you there. I branched off main to make this branch, maybe that was why.

@sgratzl sgratzl changed the base branch from main to dev December 29, 2020 13:04
@sgratzl
Copy link
Member

sgratzl commented Dec 29, 2020

I'm not sure what you want me to merge into, dev or main. When I used dev (the default), it looks like @sgratzl there was a bunch of commits from you there. I branched off main to make this branch, maybe that was why.

dev is the right one. #198 hasn't been merged yet, thus there are some commit of mine in this PR

Copy link
Member

@sgratzl sgratzl left a comment

Choose a reason for hiding this comment

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

👍

@sgratzl sgratzl merged commit 413deea into dev Dec 29, 2020
@sgratzl sgratzl deleted the blog-forecast-edit branch December 29, 2020 17:46
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