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

Remove "current" column from upload progress display #486

Merged
merged 2 commits into from
Mar 23, 2021

Conversation

jwodder
Copy link
Member

@jwodder jwodder commented Mar 23, 2021

When uploading to the new API, when the code actually reaches the point at which the upload proper happens, a "CURRENT" column appears at the right side of the pyout display containing the current number of bytes uploaded per file. As this column is not declared in the list passed to the pyout table constructor, I assume it is not supposed to be displayed in the first place. This PR removes the "current" field from the upload progress dict before yielding it to pyout, which should get rid of the column.

@jwodder jwodder added the patch Increment the patch version when merged label Mar 23, 2021
@codecov
Copy link

codecov bot commented Mar 23, 2021

Codecov Report

Merging #486 (34fe1db) into master (4c3eb5b) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #486   +/-   ##
=======================================
  Coverage   64.91%   64.91%           
=======================================
  Files          59       59           
  Lines        5991     5991           
=======================================
  Hits         3889     3889           
  Misses       2102     2102           
Flag Coverage Δ
unittests 64.91% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
dandi/upload.py 2.40% <0.00%> (ø)

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 4c3eb5b...34fe1db. Read the comment docs.

Copy link
Member

@yarikoptic yarikoptic left a comment

Choose a reason for hiding this comment

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

THANK YOU! I kept annoyed with it myself, not yet sure if there would be any side-effect. Just recommended a little change

dandi/upload.py Outdated Show resolved Hide resolved
Co-authored-by: Yaroslav Halchenko <debian@onerussian.com>
@yarikoptic yarikoptic mentioned this pull request Mar 23, 2021
@jwodder jwodder added the release Create a release when this pr is merged label Mar 23, 2021
@yarikoptic
Copy link
Member

Thank you @jwodder for the PR and a detailed description. Let's release!

@yarikoptic yarikoptic merged commit 42e80b3 into master Mar 23, 2021
@yarikoptic yarikoptic deleted the rm-current-col branch March 23, 2021 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged release Create a release when this pr is merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants