NatLabRockies / NatLabRockies/torc
Optimize checking for existing ro-crate records
@arjlai221 is already working on this.
Since Apr 21, 2026.
- Dominant language
- Rust
- Stars
- 19
- Forks
- 6
- Avg merge
- 11h 1m
- Merged PRs (30d)
- 1
Description
Copilot made this suggestion:
"find_entity_for_file calls list_ro_crate_entities (fetching all entities for the workflow) to find a single record by file_id. In create_ro_crate_entity_for_file this is called once, and in create_ro_crate_entity_for_output_file it is called once more, meaning for each file processed there is a full list fetch. For workflows with many entities this becomes an O(n) per-file lookup. Consider adding a dedicated API endpoint to look up an entity by file_id, or pass in a pre-fetched map of existing entities to these helpers."
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.