New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release candidate (#2298) #2299
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…into printable-to-utf8
replace printable for try/except utf-8
* Adding branch description * Adding spaces * Adding timeline
upgrading software
…client-to-qiita
r_client to qiita
rm moi and ipython
…-create_raw_data
…ansfer-copy-raw-data
* fix #1293 * flake8 * fix errors * addressing @ElDeveloper comments
fix awaiting_approval list bug
* replace printable for try/except utf-8 * add sleeps for tests * moving wait_for_prep_information_job * just leaving the changes on this pr * Fixing tests * Reverting changes * dist: precise * addressing @josenavas comments * fix-2212 * Adding branch description to Contributing (#2256) * Adding branch description * Adding spaces * Adding timeline * upgrading software * mv r_client to qiita * from redis import Redis * rm moi and ipython * Move qiita_db/private.py -> qiita_ware/private_plugin.py * _system_call as system_call * flake8 * rm create_raw_data * Transferring VAMPS submission to internal job * rm other unnecessary files * addressing @josenavas comment * Fixing merge conflicts * Adding tests to private plugin * rm wrapper.py * Removing import * Fixing import * Modifying GUI to use the plugin * moving qiita pet in .travis.yml * Transfer submit to vamps (#2265) * Move qiita_db/private.py -> qiita_ware/private_plugin.py * Transferring VAMPS submission to internal job * Fixing import * some other .travis.yml fixes * addressing @josenavas comment * Adding success test * adding some methods * flake8 * Transfer copy raw data (#2267) * Move qiita_db/private.py -> qiita_ware/private_plugin.py * Transferring VAMPS submission to internal job * Fixing merge conflicts * Adding tests to private plugin * Removing import * Fixing import * Modifying GUI to use the plugin * Adding success test * change imports * adding delete_artifact and create_sample_template * fixing errors and gui * fix delete error * ENH: Make jobs list modal With some recent changes to the position of the navigation bar, the header of the jobs list is cut from screen. With this patch in place, the jobs list will be shown as a modal window, so that won't be a problem anymore. * Transfer update delete templates (#2274) * Moving update_sample_template * Transfer update_sample_template * Porting update prep template * Moving delete sample or column * Removing tests * Removing dispatchable and its tests * Updating interface to use the new functionality' * Adapting the prep template GUI * Submitting jobs * Fixing tests * Removing qiita_ware/context.py * flake8ing * Fixing _system_call * Safeguarding the call to rollback * Unmasking more errors * Forcing different connections on different processes * Moving job completion to internal plugin structure * Removing unused code * Forcing the creation of a new transaction on the jobs * Fixing tests * forcing the commit * Fixing all tests * Addressing @antgonza's comments * Addressing @antgonza's comment * Addressing @ElDeveloper's comments * BUG: Fix job updates * Fixing the redis DB (#2277) * Fixing the redis DB * Addressing @antgonza's comments * redbiom install * redbiom to install_requires * mv moi-ws to qiita_websocket * init commit * addressing @wasade comment * rm webdis.log * cleaning code for initial review * install latest redbiom * fix test * ENH: Change phrasing of upload text (#2281) * ENH: Change phrasing of upload text As per a user's request. * Fix typo in docs Fixes #2259 * addressing @wasade comments and adding other tests * flake8 * addressing @josenavas comments * fix #2258 * fix #2258 * fix #858 (#2286) * addressing @ElDeveloper and @josenavas comments * @ElDeveloper :| * redbiom now adds per sample studies to analysis * jobs-list-as-modal * addresssing @josenavas comments * rm () from update_processing_job_data * add prints to review errors * rm prints * Fix 2190 (#2292) * Adding patch to add the 'name' parameter to all validate commands * Adding the parameter 'name' automatically and adding a test for it * Removing extra blank line * Adding dflt value to artifact name * Edited the wrong file * Adding user defined name at creation time * Fixing test * add is_from_analysis to artifact_handlers (#2293) * add is_from_analysis to artifact_handlers * fix test_post_metadata * fix error * WIP: rm sudo from travis * adding sed for config file * fix sed * rm & * cat redis.conf * using local redis.conf * # protected-mode yes * # supervised no * redis-server --port 7777 & * Fix 1293 (#2291) * fix #1293 * flake8 * fix errors * addressing @ElDeveloper comments * rm redis.conf * fix awaiting_approval list bug * Fix #2276 (#2294) * Fix #2276 * Factoring out generate nginx directory file list * Factoring out the nginx file list writing * Factoring out generating the file list of an artifact * Factoring out the header setting * Addressing @antgonza's comment * Addressing @wasade's comments
Codecov Report
@@ Coverage Diff @@
## master #2299 +/- ##
=========================================
+ Coverage 92.81% 93.8% +0.98%
=========================================
Files 171 163 -8
Lines 18925 18581 -344
=========================================
- Hits 17566 17430 -136
+ Misses 1359 1151 -208
Continue to review full report at Codecov.
|
* Changing study deletion message * Addressing @antgonza's comments
improving redbiom help and fixing bugs
…into delete-study-with-tags
Delete study with tags
* nice display of 504 * addressing @ElDeveloper comment
* adds EBI submission of shotgun samples * flake8 fixes * fix error * fixing comments after meeting * addressing @josenavas comments
antgonza
changed the title
[DO NOT MERGE] Release candidate (#2298)
Release candidate (#2298)
Sep 27, 2017
🎉 😄 🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
replace printable for try/except utf-8
add sleeps for tests
moving wait_for_prep_information_job
just leaving the changes on this pr
Fixing tests
Reverting changes
dist: precise
addressing @josenavas comments
fix-2212
Adding branch description to Contributing (Adding branch description to Contributing #2256)
Adding branch description
Adding spaces
Adding timeline
upgrading software
mv r_client to qiita
from redis import Redis
rm moi and ipython
Move qiita_db/private.py -> qiita_ware/private_plugin.py
_system_call as system_call
flake8
rm create_raw_data
Transferring VAMPS submission to internal job
rm other unnecessary files
addressing @josenavas comment
Fixing merge conflicts
Adding tests to private plugin
rm wrapper.py
Removing import
Fixing import
Modifying GUI to use the plugin
moving qiita pet in .travis.yml
Transfer submit to vamps (Transfer submit to vamps #2265)
Move qiita_db/private.py -> qiita_ware/private_plugin.py
Transferring VAMPS submission to internal job
Fixing import
some other .travis.yml fixes
addressing @josenavas comment
Adding success test
adding some methods
flake8
Transfer copy raw data (Transfer copy raw data #2267)
Move qiita_db/private.py -> qiita_ware/private_plugin.py
Transferring VAMPS submission to internal job
Fixing merge conflicts
Adding tests to private plugin
Removing import
Fixing import
Modifying GUI to use the plugin
Adding success test
change imports
adding delete_artifact and create_sample_template
fixing errors and gui
fix delete error
ENH: Make jobs list modal
With some recent changes to the position of the navigation bar, the
header of the jobs list is cut from screen. With this patch in place,
the jobs list will be shown as a modal window, so that won't be a
problem anymore.
Transfer update delete templates (Transfer update delete templates #2274)
Moving update_sample_template
Transfer update_sample_template
Porting update prep template
Moving delete sample or column
Removing tests
Removing dispatchable and its tests
Updating interface to use the new functionality'
Adapting the prep template GUI
Submitting jobs
Fixing tests
Removing qiita_ware/context.py
flake8ing
Fixing _system_call
Safeguarding the call to rollback
Unmasking more errors
Forcing different connections on different processes
Moving job completion to internal plugin structure
Removing unused code
Forcing the creation of a new transaction on the jobs
Fixing tests
forcing the commit
Fixing all tests
Addressing @antgonza's comments
Addressing @antgonza's comment
Addressing @ElDeveloper's comments
BUG: Fix job updates
Fixing the redis DB (Fixing the redis DB #2277)
Fixing the redis DB
Addressing @antgonza's comments
redbiom install
redbiom to install_requires
mv moi-ws to qiita_websocket
init commit
addressing @wasade comment
rm webdis.log
cleaning code for initial review
install latest redbiom
fix test
ENH: Change phrasing of upload text (ENH: Change phrasing of upload text #2281)
ENH: Change phrasing of upload text
As per a user's request.
Fixes #2259
addressing @wasade comments and adding other tests
flake8
addressing @josenavas comments
fix admins don't have access to analysis #2258
fix admins don't have access to analysis #2258
fix Single study analysis shortcut #858 (fix #858 #2286)
addressing @ElDeveloper and @josenavas comments
@ElDeveloper :|
redbiom now adds per sample studies to analysis
jobs-list-as-modal
addresssing @josenavas comments
rm () from update_processing_job_data
add prints to review errors
rm prints
Fix 2190 (Fix 2190 #2292)
Adding patch to add the 'name' parameter to all validate commands
Adding the parameter 'name' automatically and adding a test for it
Removing extra blank line
Adding dflt value to artifact name
Edited the wrong file
Adding user defined name at creation time
Fixing test
add is_from_analysis to artifact_handlers (add is_from_analysis to artifact_handlers #2293)
add is_from_analysis to artifact_handlers
fix test_post_metadata
fix error
WIP: rm sudo from travis
adding sed for config file
fix sed
rm &
cat redis.conf
using local redis.conf
protected-mode yes
supervised no
redis-server --port 7777 &
Fix 1293 (Fix 1293 #2291)
fix bulk delete #1293
flake8
fix errors
addressing @ElDeveloper comments
rm redis.conf
fix awaiting_approval list bug
Fix Trying to download a 'html summary dir' filepath type will fail #2276 (Fix #2276 #2294)
Fix Trying to download a 'html summary dir' filepath type will fail #2276
Factoring out generate nginx directory file list
Factoring out the nginx file list writing
Factoring out generating the file list of an artifact
Factoring out the header setting
Addressing @antgonza's comment
Addressing @wasade's comments