Skip to content

Fix hyper diffusion equation solver_showcase_1d.ipynb - #69

Closed
qiauil wants to merge 1 commit into
Ceyron:mainfrom
qiauil:main
Closed

Fix hyper diffusion equation solver_showcase_1d.ipynb#69
qiauil wants to merge 1 commit into
Ceyron:mainfrom
qiauil:main

Conversation

@qiauil

@qiauil qiauil commented Jun 30, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@Ceyron

Ceyron commented Oct 28, 2025

Copy link
Copy Markdown
Owner

Hi @qiauil,
Thanks for noting this, and sorry for the late reply.
I think it is a question of definition. I chose to have a negative sign here such that the coefficient of hyper-diffusion is positive (which I thought to be more intuitive). It also matches the form I chose for the APEBench paper on page 31.
But you raised a fair point, and I guess the documentation on top of this page, corresponding to this (permalink) doc file has to be adapted.

What do you think?

@Ceyron Ceyron mentioned this pull request Feb 10, 2026
@Ceyron Ceyron closed this in #79 Feb 11, 2026
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.

2 participants