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

Range of the x-coordinate for 1D Burgers data #51

Closed
functoreality opened this issue Oct 27, 2023 · 1 comment
Closed

Range of the x-coordinate for 1D Burgers data #51

functoreality opened this issue Oct 27, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@functoreality
Copy link

functoreality commented Oct 27, 2023

Thank you a lot for this benchmark dataset. In Appendix D.3 of the paper, the range of the x-axis is (0, 1) according to equation (11,12).
Screenshot_20231027_104930
However, in Fig.7, the range is (-1, 1) according to the curves, and (0, 1) according to the two-dimensional plot.
Screenshot_20231027_103936
In the configuration files for data generation, the range appears to be (-1, 1) as well:


For the dataset I downloaded:

with h5py.File('1D_Burgers_Sols_Nu0.01.hdf5', "r") as h5_file:
    print(h5_file['x-coordinate'][:])
# [-0.99902344 -0.9970703  -0.9951172  ...  0.9951172   0.9970703   0.99902344]

This seems a bit confusing to me.

Furthermore, if the range of the x-axis is (-1, 1), then I find a scaling issue similar to the Advection data mentioned here. If I have not misunderstood, for the Burgers equation, the shock propagates at a speed of (uL + uR) / 2. According to the first panel of Fig.7, the average of uL and uR is approximately -0.8, but the propagation speed seems to be doubled. I wonder whether this is related to the following lines in the data generation script:

f_upwd = (fR[1:cfg.multi.nx+2] + fL[2:cfg.multi.nx+3]
- 0.5*jnp.abs(uL[2:cfg.multi.nx+3] + uR[1:cfg.multi.nx+2])*(uL[2:cfg.multi.nx+3] - uR[1:cfg.multi.nx+2]))

In the updated script for the Advection equation, the value of the flux is multiplied by 0.5, but this operation is not applied here for the Burgers equation.

Thanks in advance for your response.

@kmario23 kmario23 added the bug Something isn't working label Nov 3, 2023
@kmario23
Copy link
Member

kmario23 commented Nov 3, 2023

Thank you for bringing this up! We have implemented the fix in the latest commit 5f19987

The new dataset and results will be updated in the next upload of the paper.

@kmario23 kmario23 closed this as completed Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants