Skip to content

Stability upgrades#127

Merged
jaykalinani merged 28 commits intoEinsteinToolkit:developmentfrom
MChabanov:cons_group
Feb 19, 2026
Merged

Stability upgrades#127
jaykalinani merged 28 commits intoEinsteinToolkit:developmentfrom
MChabanov:cons_group

Conversation

@MChabanov
Copy link
Copy Markdown
Collaborator

  • conservatives are in a group: possible to use prolongate_per_group parameter
  • ceilings on sigma and inv_beta
  • added helper grid functions
  • caps on velocity after reconstruction (with parameter v_vec)
  • minor change on lower limit of tau
  • remove unused code (related to cons_error_limit)
  • make prolongation operator and order explicit (hard-coded) in interface.ccl
  • make reconstruction methods steerable

Copy link
Copy Markdown
Collaborator

@lwJi lwJi left a comment

Choose a reason for hiding this comment

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

The cons_group part looks good to me. I'll leave the rest for other reviewers, because I'm not an expert on that.

Copy link
Copy Markdown
Collaborator

@jaykalinani jaykalinani left a comment

Choose a reason for hiding this comment

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

All looks good in general! Thanks for the effort!!
P.S. if not, I would suggest clang-formatting the edited files

Copy link
Copy Markdown
Collaborator

@acwen11 acwen11 left a comment

Choose a reason for hiding this comment

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

Other than the one comment this looks good!

@jaykalinani jaykalinani merged commit 19a463e into EinsteinToolkit:development Feb 19, 2026
3 checks passed
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.

4 participants