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

Corrected adapt functions for Grids, Models, ParameterStructs and SpectralTransform #201

Closed
wants to merge 3 commits into from

Conversation

katharinamaetschke
Copy link

I updated the adapt functions for the new Grid files in the way that Milan suggested and also uploaded the Models.jl, Parameter_structs.jl and spectral_transform.jl files again, since I closed the last PR for these.

Adapt functions for the updated Grids files.
New PR for the adapt functions for models.jl, parameter_structs.jl and spectral_transform.jl.
…Structs

Adapt functions for the newest versions of models.jl and spectral_transform.jl as well as parameter_structs.jl.
@milankl
Copy link
Member

milankl commented Dec 14, 2022

I don't know whether you want to address this in this pull request, but with #204 and other pull requests mentioned in #192 I'm still actively changing the Tendencies, GridVariables, DynamicsVariables, SurfaceVariables structs in diagnostic_variables.jl. Just as a heads up.

@milankl milankl added the gpu 🖼️ Everthing GPU related label Dec 15, 2022
@milankl
Copy link
Member

milankl commented Feb 7, 2023

@katharinamaetschke please note #241, the grids are now moved to src/RingGrids in a submodule RingGrids, similar for LowerTriangularMatrix in src/LowerTriangularMatrices and SpeedyTransforms in src/SpeedyTransforms to give everything more structure.

@milankl
Copy link
Member

milankl commented Sep 5, 2024

@maximilian-gelbrecht is this actually superceded now?

@maximilian-gelbrecht
Copy link
Member

Yes, let's close it. We have to adjust it to the new structure in another PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gpu 🖼️ Everthing GPU related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants