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

Update IC links #408

Merged
merged 12 commits into from
Jan 24, 2025
Merged

Update IC links #408

merged 12 commits into from
Jan 24, 2025

Conversation

ChunYen-Chen
Copy link
Collaborator

@ChunYen-Chen ChunYen-Chen commented Jan 8, 2025

Resolve #390.

All the ICs are now on the GAMER ythub.

Add

  • sha256sum download completeness check

Note

@ChunYen-Chen ChunYen-Chen marked this pull request as ready for review January 10, 2025 08:44
@hyschive
Copy link
Contributor

@ChunYen-Chen Please solve the conflicts.

Conflicts:
	example/test_problem/ELBDM/DiskHeating/download_ic.sh
@hyschive hyschive self-requested a review January 15, 2025 01:53
@hyschive hyschive self-assigned this Jan 15, 2025
@hyschive hyschive added enhancement test Test problems labels Jan 15, 2025
@hyschive hyschive requested a review from vivi235711 January 15, 2025 08:19
@hyschive hyschive assigned vivi235711 and unassigned hyschive Jan 15, 2025
Conflicts:
	example/test_problem/Hydro/AGORA_IsolatedGalaxy/download_ic.sh
@ChunYen-Chen
Copy link
Collaborator Author

I added sha256sum check of the downloaded file.

Copy link
Contributor

@vivi235711 vivi235711 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have downloaded all the ICs and confirmed that they are the same as the old ICs. All the sha256sum work.
In yt Hub, there are 4 ICs that are not included in this pr: FermiBubble, LSS_Zoomin, Riemann and UniformGranule.

@ChunYen-Chen
Copy link
Collaborator Author

@vivi235711 Thanks for the review. Those test problems are not merged in the GAMER yet, so the related download codes are not included in this PR.

Copy link
Contributor

@hyschive hyschive left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ChunYen-Chen @vivi235711 Thanks for the PR and review! I have only one minor comment.

@hyschive hyschive merged commit a9f8126 into gamer-project:main Jan 24, 2025
@ChunYen-Chen ChunYen-Chen deleted the ythub branch January 24, 2025 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement test Test problems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upload all runtime files to yt-hub
3 participants