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 os.rename from gensim.downloader when src and dst on different partitions #1733

Merged
merged 14 commits into from
Nov 23, 2017

Conversation

anotherbugmaster
Copy link
Contributor

Tests are working on Arch Linux now. There was a problem with new downloader.py module and tmpfs partition.

@menshikh-iv
Copy link
Contributor

Nice catch @anotherbugmaster 👍

  • remove all unrelated changes (and fix your develop branch)
  • need more complicated logic for multi partition case (shutil.move to needed partitions and after - os.rename to needed dir)

@anotherbugmaster
Copy link
Contributor Author

@menshikh-iv, all done. :)

@menshikh-iv menshikh-iv changed the title Fix arch tests Fix os.rename from gensim.downloader when src and dst on different partitions Nov 23, 2017
@menshikh-iv
Copy link
Contributor

Great job @anotherbugmaster 👍

@menshikh-iv menshikh-iv merged commit 1980109 into piskvorky:develop Nov 23, 2017
@anotherbugmaster anotherbugmaster deleted the fix_arch_tests branch November 23, 2017 05:20
VaiyeBe pushed a commit to VaiyeBe/gensim that referenced this pull request Nov 26, 2017
…rtitions (piskvorky#1733)

* Fix typo

* Make `save_corpus` private

* Annotate `bleicorpus.py`

* Make __save_corpus weakly private

* Fix _save_corpus[2]

* Document bleicorpus in Numpy style

* Fix tests on Arch Linux

* Revert "Document bleicorpus in Numpy style"

This reverts commit 04c2e61.

* Revert "Fix _save_corpus[2]"

This reverts commit 3b55037.

* Revert "Make __save_corpus weakly private"

This reverts commit 34bd9ef.

* Revert "Annotate `bleicorpus.py`"

This reverts commit 34cccfe.

* Revert "Make `save_corpus` private"

This reverts commit cde582e.

* Revert "Fix typo"

This reverts commit 3cc34ff.

* Handle the multiple partitions case
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

2 participants