Skip to content

Commit

Permalink
Fix target_type in auto-conversion context
Browse files Browse the repository at this point in the history
which means we collect and store the converted dataset as the job input.

We don't need to wait for the galaxy.json collection, we know the exact
target type already. That fixes the retrieval in
`get_converted_files_by_type`.

I believe this was always the intention and is how it works if the
dataset already exists (i.e. on a re-run), and for all converters that
don't use galaxy.json The converter records the dataset that the user
chose, so there's no gap in the provenance either.

This somewhat addresses
galaxyproject/total-perspective-vortex#141
so you can (reliably) differetiate your rules on the input datatype
and filesize combination.
  • Loading branch information
mvdbeek committed Dec 11, 2024
1 parent 616fe39 commit f10ed72
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions lib/galaxy/datatypes/data.py
Original file line number Diff line number Diff line change
Expand Up @@ -848,6 +848,7 @@ def convert_dataset(
trans, incoming=params, set_output_hid=visible, history=history, flush_job=False
)
for converted_dataset in converted_datasets.values():
converted_dataset.extension = target_type
original_dataset.attach_implicitly_converted_dataset(trans.sa_session, converted_dataset, target_type)
trans.app.job_manager.enqueue(job, tool=converter)
if len(params) > 0:
Expand Down

0 comments on commit f10ed72

Please sign in to comment.