Skip to content

Conversation

@antgonza
Copy link
Member

This PR:

  1. fixes the EBI-ENA submissions that contain samples that were submitted before
  2. updated the size of the submission that is now allowed (10M, before 8.5M)

The big change is that before we could "update" the sample information and now it's not allowed, so basically we need to skip any sample that has been previously submitted from the sample information - note that it is still added in the experiment.

@codecov-io
Copy link

codecov-io commented Apr 13, 2020

Codecov Report

Merging #2992 into dev will decrease coverage by 0.04%.
The diff coverage is 83.63%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #2992      +/-   ##
==========================================
- Coverage   95.02%   94.97%   -0.05%     
==========================================
  Files          74       74              
  Lines       14081    14124      +43     
==========================================
+ Hits        13380    13414      +34     
- Misses        701      710       +9     
Impacted Files Coverage Δ
...ita_db/metadata_template/base_metadata_template.py 96.35% <ø> (ø)
qiita_db/test/test_study.py 99.78% <ø> (ø)
qiita_db/study.py 93.86% <20.00%> (-1.45%) ⬇️
qiita_db/util.py 90.67% <92.85%> (-0.07%) ⬇️
qiita_db/analysis.py 96.39% <100.00%> (+0.01%) ⬆️
qiita_db/artifact.py 98.26% <100.00%> (+0.02%) ⬆️
qiita_db/metadata_template/prep_template.py 98.04% <100.00%> (+0.07%) ⬆️
...ta_db/metadata_template/test/test_prep_template.py 99.55% <100.00%> (+<0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c6e9ea...416246a. Read the comment docs.

Copy link
Contributor

@ElDeveloper ElDeveloper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one question

antgonza and others added 3 commits April 14, 2020 11:18
* Update processing-recommendations.rst

Adding metatranscriptome processing documentation:Sortmerna for robosomla read filtering

* Update processing-recommendations.rst

Adding metatranscriptome processing documentation:Sortmerna for ribosomal read filtering

* Update qiita_pet/support_files/doc/source/processingdata/processing-recommendations.rst

change as suggested

Co-Authored-By: Yoshiki Vázquez Baeza <yoshiki@ucsd.edu>

* Update qiita_pet/support_files/doc/source/processingdata/processing-recommendations.rst

Change as suggested

Co-Authored-By: Yoshiki Vázquez Baeza <yoshiki@ucsd.edu>

* Update qiita_pet/support_files/doc/source/processingdata/processing-recommendations.rst

Change as suggested

Co-Authored-By: Yoshiki Vázquez Baeza <yoshiki@ucsd.edu>

Co-authored-by: Yoshiki Vázquez Baeza <yoshiki@ucsd.edu>
Copy link
Contributor

@ElDeveloper ElDeveloper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a couple of comments!

@ElDeveloper ElDeveloper merged commit 1a4f17b into qiita-spots:dev Apr 27, 2020
@ElDeveloper
Copy link
Contributor

Thanks @antgonza !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants