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

TidalPy v0.6.0 #69

Merged
merged 131 commits into from
Jan 8, 2025
Merged

TidalPy v0.6.0 #69

merged 131 commits into from
Jan 8, 2025

Conversation

jrenaud90
Copy link
Owner

  • Major overhaul of RadialSolver to work with new version of CyRK.
  • Removed old TidalPy.radial_solver in favor of the new module, TidalPy.RadialSolver
  • Many quality of life improvements to RadialSolver use and results
  • Expanded RadialSolver functionality and provided user with greater control
  • Added propagation matrix technique to RadialSolver (this is still experimental)
  • Improvements to C memory allocations
  • Improved documentation
  • Many other changes throughout the whole package. Please see change log.
  • Fixes:
    • Fixed issue where radial_solver arrays could dealloc while references still pointed to them (hanging pointers). Fixes Issue RadialSolver solution gives unallocated memory for Love number results #59
    • Missing Cython compile arguments in TidalPy's utilities, nondimensional.pyx.
    • Fixed incorrect type in dynamic liquid layers that may have been causing some errors to propagate.
    • Fixed issue where TidalPy._config_path was not being updated.
    • Fixed issue where log files could not be created.

jrenaud90 added 30 commits March 5, 2024 19:22
FIX: Cython headers in nondimensional.pyx
@jrenaud90 jrenaud90 changed the title CyRK v0.6.0 TidalPy v0.6.0 Jan 7, 2025
@jrenaud90 jrenaud90 merged commit 24c2716 into main Jan 8, 2025
32 checks passed
@jrenaud90 jrenaud90 deleted the dev0.6.0 branch January 8, 2025 02:16
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.

1 participant