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

Weekly Digest (28 October, 2019 - 4 November, 2019) #2726

Closed
weekly-digest bot opened this issue Nov 4, 2019 · 0 comments
Closed

Weekly Digest (28 October, 2019 - 4 November, 2019) #2726

weekly-digest bot opened this issue Nov 4, 2019 · 0 comments

Comments

@weekly-digest
Copy link

weekly-digest bot commented Nov 4, 2019

Here's the Weekly Digest for iterative/dvc:


ISSUES

Last week 43 issues were created.
Of these, 29 issues have been closed and 14 issues are still open.

OPEN ISSUES

💚 #2721 Jupyter notebook based pipelining using DVC?, by kurianbenoy
💚 #2719 api: high level dvc.api import/export, by Suor
💚 #2711 get/import: don't assume any remotes setup in the source repo, by shcheklein
💚 #2710 import/get: improve message when we try to re-import an artifact, by shcheklein
💚 #2708 bundler/partitioner middleware, by jorgeorpinel
💚 #2706 Allow restriction dvc pull to targets, by Titousensei
💚 #2701 External output to encrypted s3 fails with ETag mismatch, by Titousensei
💚 #2699 add: improve output, extra progress is not needed in most cases, by shcheklein
💚 #2697 Using dvc only for dataset management (e.g. no dvc run pipeline)., by JoeyCarson
💚 #2696 update: improve output when import stage is fixed to an unchanged rev?, by jorgeorpinel
💚 #2691 import: importing directories output is ugly, confusing, not informative, by shcheklein
💚 #2689 output: cache metadata, by mroutis
💚 #2685 pull: wrong warning on pulling import stage, by shcheklein
💚 #2683 s3: ignore empty directories while walking files, by mroutis

CLOSED ISSUES

❤️ #2723 fix PR template to mention recommendatory nature of CC and DS, by shcheklein
❤️ #2722 Added package manager to dvc version output, by kss682
❤️ #2720 Added package manager to dvc version output., by kss682
❤️ #2718 removed cask error, by johntharian
❤️ #2717 typo changed for existance->existence, by johntharian
❤️ #2716 typos corrected, by eshaan-jeckyll
❤️ #2715 Fixed issue #2680 and Simple typo rectified, by ATR-oCiTy
❤️ #2714 Rectified simple typos, by Pandemic179
❤️ #2713 fix typo(correct spelling of existence), by GeorgeSabu-mec
❤️ #2712 fixed typo, by the-thirteenth-star
❤️ #2709 sftp: make move across drives possible, by mroutis
❤️ #2707 perf: refactor URLInfo to not use ParseResult inside, by Suor
❤️ #2705 get-url: fix NPE when src non local and dst not absolute, by shcheklein
❤️ #2704 Unexpected error when cache and artifacts are not on the same partition on SSH, by mslapek
❤️ #2703 Remove multi-threaded prefix before progress bar(#2686), by kurianbenoy
❤️ #2702 Problems when downloading file from s3, by petrochenko-pavel-a
❤️ #2700 remote: fail if does not exist, by nkrot
❤️ #2698 Verify if get_mtime_and_size can take both Path objects and strings, by algomaster99
❤️ #2695 tests: preserve the same ETag across buckets during copy, by mroutis
❤️ #2694 dvc: fix tqdm dep, by Suor
❤️ #2693 pull, at import stage: replace warning with info report, by sadielbartholomew
❤️ #2692 failing while pulling imported directories, by shcheklein
❤️ #2690 Can't find documentation for calling dvc from python module., by JoeyCarson
❤️ #2688 lock: add a workaround for a flufl.lock bug, by efiop
❤️ #2687 dvc run broken in 0.66.1. Fails with error unexpected error., by MikkelAntonsen
❤️ #2686 ui: remove multi threaded prefix before each progress bar, by shcheklein
❤️ #2684 Error on first-time dvc pull, by kevlar1818
❤️ #2682 Remove --all-commits options from usage output, by sagunji
❤️ #2681 gc: remove --all-commits options from usage output, by jorgeorpinel

LIKED ISSUE

👍 #2719 api: high level dvc.api import/export, by Suor
It received 👍 x2, 😄 x0, 🎉 x0 and ❤️ x1.

NOISY ISSUE

🔈 #2697 Using dvc only for dataset management (e.g. no dvc run pipeline)., by JoeyCarson
It received 21 comments.


PULL REQUESTS

Last week, 25 pull requests were created, updated or merged.

UPDATED PULL REQUEST

Last week, 4 pull requests were updated.
💛 #2683 s3: ignore empty directories while walking files, by mroutis
💛 #2679 updater: detect conda, by n3hrox
💛 #2669 Sort dvc pipeline list, by SrividyaKK
💛 #2551 GDrive remote support, by MaxRis

MERGED PULL REQUEST

Last week, 21 pull requests were merged.
💜 #2722 Added package manager to dvc version output, by kss682
💜 #2717 typo changed for existance->existence, by johntharian
💜 #2716 typos corrected, by eshaan-jeckyll
💜 #2715 Fixed issue #2680 and Simple typo rectified, by ATR-oCiTy
💜 #2714 Rectified simple typos, by Pandemic179
💜 #2713 fix typo(correct spelling of existence), by GeorgeSabu-mec
💜 #2712 fixed typo, by the-thirteenth-star
💜 #2709 sftp: make move across drives possible, by mroutis
💜 #2707 perf: refactor URLInfo to not use ParseResult inside, by Suor
💜 #2705 get-url: fix NPE when src non local and dst not absolute, by shcheklein
💜 #2703 Remove multi-threaded prefix before progress bar(#2686), by kurianbenoy
💜 #2698 Verify if get_mtime_and_size can take both Path objects and strings, by algomaster99
💜 #2695 tests: preserve the same ETag across buckets during copy, by mroutis
💜 #2694 dvc: fix tqdm dep, by Suor
💜 #2693 pull, at import stage: replace warning with info report, by sadielbartholomew
💜 #2692 failing while pulling imported directories, by shcheklein
💜 #2688 lock: add a workaround for a flufl.lock bug, by efiop
💜 #2682 Remove --all-commits options from usage output, by sagunji
💜 #2674 logger: output footer once for multithreaded push/pull errors, by n3hrox
💜 #2673 Write test for get_inode, by algomaster99
💜 #2644 dvc: change ` for ' in dynamic parts of output messages, by jorgeorpinel


COMMITS

Last week there were 64 commits.
🛠️ fix PR template to mention recommendatory nature of CC and DS by shcheklein
🛠️ dvc: bump to 0.66.4 by efiop
🛠️ test: try removing get_url test by efiop
🛠️ test: fix formatting by efiop
🛠️ test: get-url: try using with mock_s3() instead of a decorator by efiop
🛠️ Merge pull request #2674 from n3hrox/enhance/footnote_deduplication logger: output footer once for multithreaded push/pull errors by efiop
🛠️ revert print to logger by n3hrox
🛠️ move footer to better place and change logging to stderr by n3hrox
🛠️ test: s3: try using boto3.client by efiop
🛠️ Merge pull request #2722 from kss682/2672_1 Added package manager to dvc version output by efiop
🛠️ setup: freeze awscli version by efiop
🛠️ test: freeze moto version by efiop
🛠️ test: get-url: don't use get_aws_url by efiop
🛠️ Merge pull request #2715 from ATR-oCiTy/master Fixed issue #2680 and Simple typo rectified by efiop
🛠️ Merge pull request #2716 from eshaan-jeckyll/master typos corrected by efiop
🛠️ Merge pull request #2717 from johntharian/master typo changed for existance->existence by efiop
🛠️ Merge pull request #2714 from Pandemic179/master Rectified simple typos by efiop
🛠️ Merge pull request #2713 from GeorgeSabu-mec/master fix typo(correct spelling of existence) by efiop
🛠️ Merge pull request #2712 from the-thirteenth-star/master fixed typo by efiop
🛠️ typo changed for existance->existaence by johntharian
🛠️ fix typo(correct spelling of existence) by GeorgeSabu-mec
🛠️ Update fetch.py by the-thirteenth-star
🛠️ Rectified install: Cask 'dvc' is not installed. error #2680 by ATR-oCiTy
🛠️ Simple typos mectified by ATR-oCiTy
🛠️ typos corrected by eshaan-jeckyll
🛠️ Rectified simple typos by Pandemic179
🛠️ Merge pull request #2703 from kurianbenoy/master Remove multi-threaded prefix before progress bar(#2686) by efiop
🛠️ Merge pull request #2698 from algomaster99/test-argument-types-get_mtime_and_size Verify if get_mtime_and_size can take both Path objects and strings by efiop
🛠️ Remove unnecessary wrapping of path by algomaster99
🛠️ Modify test to check for directory as well by algomaster99
🛠️ TqdmThreadPoolExecutor => ThreadPoolExecutor - Fixes https://github.com/iterative/dvc/pull/2703#discussion_r341763150 by casperdcl
🛠️ remove special TqdmThreadPoolExecutor logic - fixes #2686 - closes #2703 by casperdcl
🛠️ Change import order in progress file by kurianbenoy
🛠️ Remove multi-threaded prefix before progress bar(#2686) by kurianbenoy
🛠️ Merge pull request #2709 from mroutis/fix-2704 sftp: make move across drives possible by efiop
🛠️ Merge pull request #2707 from Suor/urlinfo perf: refactor URLInfo to not use ParseResult inside by efiop
🛠️ sftp: move -> use atomic copying by mroutis
🛠️ dvc: bump to 0.66.3 by efiop
🛠️ Merge pull request #2705 from shcheklein/fix-2702 get-url: fix NPE when src non local and dst not absolute by efiop
🛠️ Merge pull request #2693 from sadielbartholomew/i2685-pull-cmd-output pull, at import stage: replace warning with info report by efiop
🛠️ Replace path with path_info by algomaster99
🛠️ sftp: move -> fallback to copy when sftp.rename fails by mroutis
🛠️ sftp: make move an atomic operation by mroutis
🛠️ sftp: make move across drives possible Fix #2704 by mroutis
🛠️ fix tests by n3hrox
🛠️ local output: address PR comment, fix style a bit by shcheklein
🛠️ perf: refactor URLInfo to not use ParseResult inside This permites URLInfo construction from parts without building/reparsing string, which makes it around 20% faster. Code is also simpler and a bit shorter now. Other improvements: - added URLInfo.replace(path=...) - URLInfo._path now stringifies properly - protected against passing path not starting with / - do not allow query, params, fragments and passwords in urls (they were silently ignored previously) by Suor
🛠️ fix typo upload -> download by n3hrox
🛠️ move footer from logger to main by n3hrox
🛠️ remove info message by efiop
🛠️ fix order by efiop
🛠️ fix formatting by efiop
🛠️ CLI: change return output for 'checkout' and 'pull' commands Change messages output after return of some commands: * Remove warning for 'dvc checkout' & commands calling it under the hood * Add an informational message output for 'dvc pull' command Fixes #2685. by sadielbartholomew
🛠️ get-url: fix NPE when src non local and dst not absolute by shcheklein
🛠️ Merge pull request #2695 from mroutis/test-s3-etag tests: preserve the same ETag across buckets during copy by efiop
🛠️ Test acceptable path types of get_mtime_and_size by algomaster99
🛠️ tests: same ETag across buckets by mroutis
🛠️ dvc: bump to 0.66.2 by efiop
🛠️ [failing while pulling imported directories (#2692) * pull: fix-2684 - failing while pulling imported directories

  • pull: fix-2684 - fix code indentation

  • minor imported directories pull test update

Co-Authored-By: Ruslan Kuprieiev kupruser@gmail.com](2ec0aa0) by shcheklein
🛠️ dvc: fix tqdm dep (#2694) tqdm==4.35.0 was throwing KeyError: 'percentage'. by Suor
🛠️ [lock: add a workaround for a flufl.lock bug (#2688) This bug doesn't affect us during regular dvc CLI commands, but is not
visualy pleasing during API usage.

Fixes #2573](877e385) by efiop
🛠️ [dvc: change for ' in dynamic parts of output messages (#2644) * dvc: change for ' in dinamic parts of DVC outputs

per #2644 (comment)

  • dvc: use ' for dynamic output and ` for DVC names

per https://github.com/iterative/dvc/pull/2644#discussion_r338417698](https://github.com/iterative/dvc/commit/a48481b01fd607b36f9ea5b643af24c938254f60) by jorgeorpinel
🛠️ [Remove --all-commits options from usage output (#2682) * Comment out --all-commits optional arg


CONTRIBUTORS

Last week there were 17 contributors.
👤 shcheklein
👤 efiop
👤 n3hrox
👤 johntharian
👤 GeorgeSabu-mec
👤 the-thirteenth-star
👤 ATR-oCiTy
👤 eshaan-jeckyll
👤 Pandemic179
👤 algomaster99
👤 casperdcl
👤 kurianbenoy
👤 mroutis
👤 Suor
👤 sadielbartholomew
👤 jorgeorpinel
👤 sagunji


STARGAZERS

Last week there were 45 stagazers.
neesonqk
LeeSuper666
WIN32GG
EmperorKaiser
BernMcCarty
natevecc
merqurio
NickMinnellaCS96
aabramovrepo
TariqAHassan
edubxb
davenisarg
robert780612
sara-02
MakarTroyan
filipetheodorodocket
theikkila
sadielbartholomew
valeriocardoso
b-luu
aerubanov
brianluby
shaowei-su
raghuram-kukun
tobym
MrTwoFace
mfrr
iwanbolzern
chcdc
jashshah
e3bo
shrutix
walshie4
jpambrun
kaweantonio
dbbert
kss682
imbsky
prateekj117
pEight
jbarmash
Pandemic179
albertodicagno
tejashah88
kojino
You all are the stars! 🌟


RELEASES

Last week there were 3 releases.
🚀 0.66.4 0.66.4
🚀 0.66.3 0.66.3
🚀 0.66.2 0.66.2


That's all for last week, please 👀 Watch and Star the repository iterative/dvc to receive next weekly updates. 😃

You can also view all Weekly Digests by clicking here.

Your Weekly Digest bot. 📆

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

No branches or pull requests

1 participant