Skip to content
This repository was archived by the owner on Oct 16, 2025. It is now read-only.

Add option to scramble a study for saving and sharing unpublished results#274

Open
olgabot wants to merge 1905 commits intomasterfrom
save_scrambled
Open

Add option to scramble a study for saving and sharing unpublished results#274
olgabot wants to merge 1905 commits intomasterfrom
save_scrambled

Conversation

@olgabot
Copy link
Collaborator

@olgabot olgabot commented Mar 13, 2015

Add option to study.save to save the current data and shuffle the gene and sample IDs so the biological signal is lost. For saving datapackages for debugging purposes, so users can submit datapackages without worrying about losing published results

  • Is it mergable?
  • Did it pass the tests?
  • If it introduces new functionality in scripts/ is it tested?
    Check for code coverage. To run code coverage on only the file you changed,
    for example flotilla/compute/splicing.py, use this command:
    py.test --cov flotilla/compute/splicing.py --cov-report term-missing flotilla/test/compute/test_splicing.py
    which will show you which lines aren't covered by the tests.
  • Do the new functions have descriptive
    numpydoc
    style docstrings?
  • If it adds a new plot, is it documented in the gallery?
  • Is it well formatted? Look at make pep8 and make lint output
  • Is it documented in the doc/releases/?
  • Was a spellchecker run on the source code and documentation after
    changes were made?

* 'master' of github.com:YeoLab/flotilla: (31 commits)
  Add GO to release notes
  fix bug addded while linting
  fix pep8
  remove plot_study_sample_legend because it's unused
  Fix lint errors because pyflakes got majorly strict
  remove go.py (replaced with the data model)
  Fix tests and add more for  a total of 98% coverage!
  add example gene ontology data
  Add tests for gene ontology data
  Sort dataframe by p value
  Make GeneOntologyData accessible from base flotilla
  also check that features of interest overlap with GO term
  make sure provided domains are converted to frozenset, and fix logic for checking domain of GO term
  access 0th value because 0th item by series may not be valid
  better error message
  also say error for why species data wasn't loaded
  df.empty is attribute, not method
  make sure to create defaultdict properly
  Add go term domains and name to growing defaultdict, and return None if no GO categories were enriched
  stdout --> warnings
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments