Skip to content

Conversation

@msimberg
Copy link
Contributor

Attempting to fix data downloads after #1002.

With my current understanding of how the downloads work, the download and extraction should happen in the "root" directory (_ranked_data_path), but the tarball actually contains subdirectories which match the experiment-specific subdirectory (which is what we check for to see if the data already exists).

I've tested re-downloading one grid and that worked. Please test it on your local setups to see I haven't messed something up.

Copy link
Contributor

@havogt havogt left a comment

Choose a reason for hiding this comment

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

Thanks for the fix! With the suggested change, my use-case would be covered as well. I tested a few cases and seems good.

@havogt
Copy link
Contributor

havogt commented Jan 29, 2026

Note, unrelated to this PR I opened #1021 which might result in weird errors if you run into the problem.

@havogt
Copy link
Contributor

havogt commented Jan 29, 2026

In #1004 I would consider taring without subdirectories which gives more flexibility, and we can move back to the simpler version without subdir.

Copy link
Contributor

@jcanton jcanton left a comment

Choose a reason for hiding this comment

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

looks good to me, also tested locally and works

@jcanton
Copy link
Contributor

jcanton commented Jan 29, 2026

In #1004 I would consider taring without subdirectories which gives more flexibility, and we can move back to the simpler version without subdir.

ok, but I can make this change in PR1004, not here
(moreover, if we include the NAMELIST dump so we can improve the experiment config (like I did now in #1004), it could still make sense to have the actual data separate in its own subdir as it is now)

@msimberg msimberg requested review from havogt and jcanton January 30, 2026 10:42
Copy link
Contributor

@jcanton jcanton left a comment

Choose a reason for hiding this comment

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

LGTM.2 ;-)

Copy link
Contributor

@havogt havogt left a comment

Choose a reason for hiding this comment

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

Tested and works (including the muphys use-case).

@github-actions
Copy link

Mandatory Tests

Please make sure you run these tests via comment before you merge!

  • cscs-ci run default
  • cscs-ci run distributed

Optional Tests

To run benchmarks you can use:

  • cscs-ci run benchmark-bencher

To run tests and benchmarks with the DaCe backend you can use:

  • cscs-ci run dace

To run test levels ignored by the default test suite (mostly simple datatest for static fields computations) you can use:

  • cscs-ci run extra

For more detailed information please look at CI in the EXCLAIM universe.

@havogt havogt merged commit 6b513ec into C2SM:main Jan 30, 2026
46 checks passed
@msimberg msimberg deleted the fix-data-download branch January 30, 2026 14:06
jcanton added a commit that referenced this pull request Jan 30, 2026
* main:
  Fix data download paths (#1020)
  Combine divergence damping coefficient calculation with other stencils (#951)
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.

3 participants