Skip to content
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

Fix 1951 #1955

Merged
merged 3 commits into from
Sep 29, 2016
Merged

Fix 1951 #1955

merged 3 commits into from
Sep 29, 2016

Conversation

antgonza
Copy link
Member

This PR is based on #1948 as we are changing the same files, please review/merge that one first.

The error was in how we were calculating the "all diff values" in the GUI, the actual stats summary is fine.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 84.012% when pulling 9ca2437 on antgonza:fix-1951 into 5ab58d4 on biocore:master.

@ElDeveloper
Copy link
Member

@antgonza, whenever you have a chance, there's conflicts in this PR, would you mind updating this branch?

@antgonza
Copy link
Member Author

@ElDeveloper I was working on that when your message came in ... 😄

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 84.008% when pulling 50e65f2 on antgonza:fix-1951 into 8f65432 on biocore:master.

@ElDeveloper
Copy link
Member

Wonderful @antgonza! 👍

@josenavas josenavas merged commit 94595c6 into qiita-spots:master Sep 29, 2016
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.

None yet

4 participants