Skip to content
Snippets Groups Projects

feat: add parameter class

Merged Michele Garioni requested to merge feature into main
2 unresolved threads
Edited by Alex Kanitz

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
28 rna_sd_length: RNA fragment length standard deviation.
29 read_length: Read length.
30 intron_rate: Constant probability of retaining an intron.
31 add_poly_a: Boolean option to add a poly A tail.
32 poly_a_func: Function to add a poly_a tail.
33 primer_seq: Sequence of the primer.
34 priming_func: Function that evaluates internal priming.
35 """
36
37 def __init__(self, param_file: Path) -> None:
38 """Class constructor."""
39 self.param_file: Path = Path(param_file)
40 with open(param_file) as f:
41 LOG.info("Loading parameters...")
42 for line in f:
43 s = line.split(':')
  • Afterwards you mention CSV, which is comma-separated file. Probably it would make sense to parameterize this, so pass a second param separator to the class, then set the separator from there (you can use as default whatever is in our test file, not sure if ,, : or tab)

  • Please register or sign in to reply
  • Alex Kanitz
  • 29 read_length: Read length.
    30 intron_rate: Constant probability of retaining an intron.
    31 add_poly_a: Boolean option to add a poly A tail.
    32 poly_a_func: Function to add a poly_a tail.
    33 primer_seq: Sequence of the primer.
    34 priming_func: Function that evaluates internal priming.
    35 """
    36
    37 def __init__(self, param_file: Path) -> None:
    38 """Class constructor."""
    39 self.param_file: Path = Path(param_file)
    40 with open(param_file) as f:
    41 LOG.info("Loading parameters...")
    42 for line in f:
    43 s = line.split(':')
    44 if s[0] == 'Csv transcripts file':
  • Going to merge this. If we end up using it, we can still take care of the open issues.

  • Alex Kanitz approved this merge request

    approved this merge request

  • Alex Kanitz changed title from Feature to feat: add parameter class

    changed title from Feature to feat: add parameter class

  • merged

  • Please register or sign in to reply
    Loading