Repository navigation
Conversation
ObadaS
marked this pull request as ready for review
August 28, 2026 07:22
ObadaS
force-pushed
the
compute_worker_reference_data
branch
from
September 1, 2026 12:54
27c3515 to
a31df34
Compare
Didayolo
self-requested a review
September 8, 2026 12:25
Member
|
Note: test that the folder is not accessible during ingestion |
ObadaS
force-pushed
the
compute_worker_reference_data
branch
4 times, most recently
from
September 14, 2026 13:51
332aee4 to
b992bef
Compare
ObadaS
force-pushed
the
compute_worker_reference_data
branch
2 times, most recently
from
September 29, 2026 09:26
e77d1b3 to
0e2a05c
Compare
ObadaS
force-pushed
the
compute_worker_reference_data
branch
2 times, most recently
from
October 8, 2026 07:17
78b4cfc to
07d452a
Compare
Didayolo
force-pushed
the
compute_worker_reference_data
branch
from
October 9, 2026 13:40
07d452a to
c952847
Compare
Member
Change (fix I made)Line 526 self.data_dir = os.path.join(Settings.HOST_DIRECTORY, "data") # absolute path to data in the host
self.reference_dir = os.path.join(Settings.BASE_DIR, "reference") # path inside the worker containerInstead of: self.data_dir = os.path.join(Settings.HOST_DIRECTORY, "data") # absolute path to data in the host
self.reference_dir = os.path.join(Settings.HOST_DIRECTORY, "reference")Why
Line 1283 # Reference data, only for the scoring phase.
volumes_host.extend([self._get_host_path(self.reference_dir)])While Line 1262 volumes_host = [
self._get_host_path(program_dir),
self._get_host_path(self.output_dir),
self.data_dir,
self._get_host_path(self.root_dir, "submission")
]By the way, we could update the code here (line 1262) to convert |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Add a folder to put reference data in on the compute worker, similar to
/codabench/data. This folder will only be mounted during the scoring phase.Checklist