Convenience functions maybe needed

Open
#39 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
45/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Stale
Tech stack
r
Domain
data

Research direction

Locate the existing yaml_path discovery, get_entity_name, and the current path-building code. Check how entity TSV and YAML paths are used internally. Done means the requested convenience functions exist, the existing YAML discovery is refactored as described, and internal callers use the shared helpers.

Written by the indexing model from the issue text.

Description

enhancement
  • get_entity_tsv_path(entity, output_directory) (returns glue("{output_dir}/entity-{get_entity_name(entity)}.tsv")) - use it internally too
  • probably also the yaml equivalent
  • maybe refactor existing yaml_path discovery into discover_yaml_path(entity, output_directory)
Dominant language
R
Stars
0
Forks
0
Avg merge
2d 1h
Merged PRs (30d)
1

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from VEuPathDB/study-wrangler

All issues in VEuPathDB/study-wrangler

Similar issues

More R issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.