Skip to content
Snippets Groups Projects

feat: add function to calculate mean and variance

Closed Reto Tschannen requested to merge issue12 into main

Add function to calculate mean and variance from a directory with files of gene expression(Work in progress)

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
  • Alex Kanitz
  • Alex Kanitz
  • Alex Kanitz
  • Alex Kanitz
  • Alex Kanitz
  • Alex Kanitz
  • Alex Kanitz
  • Alex Kanitz
    • Apart from the comments above, the CI pipeline failed with linter errors and there are no unit tests. Please fix linter and other CI errors, add unit tests that cover all of your code.

    • I can't resolve these flake8 issues, how can I fix them (My local flake8 shows no problems, but when I push, the qc fails because of them)?

      $ flake8 src/ setup.py
      src/meanvariancefunction.py:12:1: D205 1 blank line required between summary line and description
      src/meanvariancefunction.py:12:1: D415 First line should end with a period, question mark, or exclamation point
      src/meanvariancefunction.py:12:1: D411 Missing blank line before section

      Best, Reto

      Edited by Reto Tschannen
    • Hi Reto Did you follow the instructions you actually get from running flake8 in the CI? You seem to miss some blank lines, punctuation marks, etc. You need a summary line (one line, ending with period, ? or !) then a blank line and then description.

    • I could resolve it now, but the last line was hard to locate, since it didn't show locally. Now only the tests are missing.

    • Please register or sign in to reply
  • Alex Kanitz mentioned in merge request !16 (closed)

    mentioned in merge request !16 (closed)

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading