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

errors in diffusive_ice_column_model.jl #33

Open
francispoulin opened this issue Aug 3, 2024 · 2 comments
Open

errors in diffusive_ice_column_model.jl #33

francispoulin opened this issue Aug 3, 2024 · 2 comments

Comments

@francispoulin
Copy link

diffusive_ice_column_model.jl defines two functions to be functions of (x, y, t), but the right hand side only depends on t and this causes an error.

In the current state of affairs, I think we need make the following changes:

air_ice_temperature(x, y, t) = top_T_slope * t + top_T_amplitude * sin(2π*t/day) + initial_air_ice_temperature

should become

air_ice_temperature(t) = top_T_slope * t + top_T_amplitude * sin(2π*t/day) + initial_air_ice_temperature

and

ice_ocean_temperature(x, y, t) = bottom_T_slope * t + initial_ice_ocean_temperature

should become

ice_ocean_temperature(t) = bottom_T_slope * t + initial_ice_ocean_temperature

With these changes plotting needs to be changes slightly in lines 42, 43, 189, and 190.

Do people agree?

@glwagner
Copy link
Member

glwagner commented Aug 5, 2024

What matters is that the grid is Flat in x, y:

grid = RectilinearGrid(size=20, z=(-1, 0), topology=(Flat, Flat, Bounded))

Since it is, then I agree the boundary conditions should be functions of t only.

@glwagner
Copy link
Member

glwagner commented Aug 5, 2024

PS, for code blocks, use triple backticks "```", rather than just a single backtick "`"

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

No branches or pull requests

2 participants