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

(Try 2) Improve the explanation surrounding "virtual temperature" and a hypothetical "density temperature" #174

Merged
merged 1 commit into from
Jan 12, 2024

Conversation

glwagner
Copy link
Member

@glwagner glwagner commented Jan 5, 2024

Supercedes #171, trying again now that I have contributor privileges.

From #171:

I thought the description could be improved by being a little more explicit. If I've made a mistake, let me know and I can fix it. I also embellished the "side point" by putting it in a box to emphasize that this is additional information that isn't core to understanding the main thread.

Copy link

codecov bot commented Jan 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8de5393) 93.02% compared to head (2b97107) 93.02%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #174   +/-   ##
=======================================
  Coverage   93.02%   93.02%           
=======================================
  Files           9        9           
  Lines        1133     1133           
=======================================
  Hits         1054     1054           
  Misses         79       79           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@glwagner
Copy link
Member Author

glwagner commented Jan 5, 2024

@charleskawczynski looks like buildkite won't run for this either

@charleskawczynski charleskawczynski added the Launch Buildkite Add label to launch Buildkite label Jan 5, 2024
@charleskawczynski
Copy link
Member

@glwagner, we have a label to launch buildkite in thermodynamics, looks like it's started now 🎉

Copy link
Member

@charleskawczynski charleskawczynski left a comment

Choose a reason for hiding this comment

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

Changes look fine to me.

@glwagner
Copy link
Member Author

glwagner commented Jan 6, 2024

@glwagner, we have a label to launch buildkite in thermodynamics, looks like it's started now 🎉

got it

@glwagner glwagner added this pull request to the merge queue Jan 6, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 6, 2024
@glwagner glwagner added this pull request to the merge queue Jan 6, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 6, 2024
@charleskawczynski
Copy link
Member

I'll go ahead and rebase this PR @glwagner

@charleskawczynski charleskawczynski added Launch Buildkite Add label to launch Buildkite and removed Launch Buildkite Add label to launch Buildkite labels Jan 12, 2024
@charleskawczynski
Copy link
Member

I think we were experimenting with the buildkite settings, and CI was not working because of that. Hopefully things should be smooth moving forward.

@charleskawczynski charleskawczynski added Launch Buildkite Add label to launch Buildkite and removed Launch Buildkite Add label to launch Buildkite labels Jan 12, 2024
@charleskawczynski charleskawczynski added this pull request to the merge queue Jan 12, 2024
Merged via the queue into main with commit d73a834 Jan 12, 2024
20 checks passed
@glwagner glwagner deleted the glw/docs-improvement branch January 16, 2024 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Launch Buildkite Add label to launch Buildkite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants