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

P3 fall speed regime docs #191

Closed
wants to merge 2 commits into from
Closed

P3 fall speed regime docs #191

wants to merge 2 commits into from

Conversation

rorlija1
Copy link
Contributor

Purpose

To-do

  • see the many TODOs in the docs!
  • and a little note in the P3SchemePlots.jl about parameters I haven't computed

Content

  • draft of docs
  • some parameters listed in P3SchemePlots.jl

  • I have read and checked the items on the review checklist.

@rorlija1 rorlija1 changed the title A start on P3 fall speed regime P3 fall speed regime docs Aug 25, 2023
@codecov
Copy link

codecov bot commented Aug 25, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (0519c40) 98.19% compared to head (3cda712) 98.19%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #191   +/-   ##
=======================================
  Coverage   98.19%   98.19%           
=======================================
  Files          13       13           
  Lines         887      887           
=======================================
  Hits          871      871           
  Misses         16       16           
Files Changed Coverage Δ
src/Parameters.jl 53.84% <ø> (ø)

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

@trontrytel trontrytel self-assigned this Sep 7, 2023
@trontrytel trontrytel linked an issue Sep 7, 2023 that may be closed by this pull request
@trontrytel
Copy link
Member

Closing this one. We will try to use the Chen 2022 velocities

@trontrytel trontrytel closed this Feb 29, 2024
@rorlija1 rorlija1 deleted the ro/velocitydocs branch August 12, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compute the mass and number weighted terminal velocity
2 participants