Skip to content

Commit c8bae2c

Browse files
committed
Bug fix: Convert test orbit ID from pyarrow StringScalar to a python string when creating a test orbit directory path
1 parent 7fc9d1a commit c8bae2c

File tree

2 files changed

+6
-14
lines changed

2 files changed

+6
-14
lines changed

thor/main.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,9 @@ def initialize_test_orbit(
4747
Initialize the test orbit by saving it to disk if a working directory is provided.
4848
"""
4949
if working_dir is not None:
50-
test_orbit_directory = pathlib.Path(working_dir, "inputs", test_orbit.orbit_id)
50+
test_orbit_directory = pathlib.Path(
51+
working_dir, "inputs", test_orbit.orbit_id[0].as_py()
52+
)
5153
test_orbit_directory.mkdir(parents=True, exist_ok=True)
5254
test_orbit_path = os.path.join(test_orbit_directory, "test_orbit.parquet")
5355
test_orbit.to_parquet(test_orbit_path)
@@ -113,7 +115,7 @@ def link_test_orbit(
113115
if working_dir is not None:
114116
working_dir_path = pathlib.Path(working_dir)
115117
logger.info(f"Using working directory: {working_dir}")
116-
test_orbit_directory = pathlib.Path(working_dir_path, test_orbit.orbit_id)
118+
test_orbit_directory = pathlib.Path(working_dir, test_orbit.orbit_id[0].as_py())
117119
test_orbit_directory.mkdir(parents=True, exist_ok=True)
118120
inputs_dir = pathlib.Path(working_dir_path, "inputs")
119121
inputs_dir.mkdir(parents=True, exist_ok=True)

thor/tests/test_main.py

+2-12
Original file line numberDiff line numberDiff line change
@@ -232,12 +232,7 @@ def test_link_test_orbit(
232232
else:
233233
integration_config.max_processes = 1
234234

235-
(
236-
test_orbit,
237-
observations,
238-
obs_ids_expected,
239-
integration_config,
240-
) = setup_test_data(
235+
(test_orbit, observations, obs_ids_expected, integration_config,) = setup_test_data(
241236
object_id, orbits, observations, integration_config, max_arc_length=14
242237
)
243238

@@ -264,12 +259,7 @@ def test_benchmark_link_test_orbit(
264259
else:
265260
integration_config.max_processes = 1
266261

267-
(
268-
test_orbit,
269-
observations,
270-
obs_ids_expected,
271-
integration_config,
272-
) = setup_test_data(
262+
(test_orbit, observations, obs_ids_expected, integration_config,) = setup_test_data(
273263
object_id, orbits, observations, integration_config, max_arc_length=14
274264
)
275265

0 commit comments

Comments
 (0)