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

Skip extrapolation of missing core profiles #76

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

eldond
Copy link
Collaborator

@eldond eldond commented Jul 23, 2024

  1. Make a list of profiles to extrapolate and loop through the list.
  2. Test to see whether extrapolation is worth trying or if data are missing.
    • Avoids total failure when operating on reduced samples (sometimes we make file size smaller by deleting select fields from output; we don't need everything for every test)
  3. Keep track of which profiles actually get extrapolated instead of skipped.
  4. Report results, as in X/Y profiles were extrapolated.

@eldond eldond added the enhancement New feature or request label Jul 23, 2024
@eldond eldond self-assigned this Jul 23, 2024
@eldond eldond changed the title WIP: Skip extrapolation of missing core profiles Skip extrapolation of missing core profiles Jul 23, 2024
Copy link
Collaborator

@anchal-physics anchal-physics left a comment

Choose a reason for hiding this comment

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

Looks good.

@anchal-physics anchal-physics merged commit d399c4f into master Jul 23, 2024
3 checks passed
@eldond eldond deleted the skip_missing_prof branch July 24, 2024 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants