-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
merge dev for release 0.9.7 #195
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…refactored function
* Bump pylint from 2.15.10 to 2.16.2 Bumps [pylint](https://github.com/PyCQA/pylint) from 2.15.10 to 2.16.2. - [Release notes](https://github.com/PyCQA/pylint/releases) - [Commits](pylint-dev/pylint@v2.15.10...v2.16.2) --- updated-dependencies: - dependency-name: pylint dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * fix warnings and new error * add python 3.11 * add doc in script to run tests * remove 3.10 and 3.11 tests for Windows since pip install fail --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* add missing with_hyphen_split argument in __call__ * reworked DataCleaning into a more flexible approach * fix tests, improve doc and add more pre-processors
* minor improvements to doc, code and tests * add multithreads tests in worker and fixed missing .to removal * improve contributing and changelog
* improve error handling * improve error handling with retrain model and test a test case with new error raise if no metadata * fix tests and improve new fix with a better getter * fix error message and improve tests * fix pylint * fix skipif to allow runner to run tests and fix call to prepare pretrained in locality of test to allow runner to run tests * add fake ckpt creation in not integration tests * fix setup class related to pre trained init * fix setupclass error in some tests
* merge lengths as vector to lengths as list * fix tests * fix tests * fix tests * fix tests * fix tests * minor doc corrections * fix pylint * fix doc
* add first version of uri implementation using s3path * changed to cloudpath since support more s3 provider * improve cloudpathlib handling and doc * add fn to handle weights download over URI, add new dependencies, add tests and fix setup.py * add code to upload wieghts properly, add some tests, add model version release * add release version * fixed tests, add changelog and start docs * add examples * fix changelog typos, fix parse and testing tests and code, improve cli doc and add integration test uri * fix tests and fix error in tool infer model type * fix error in cli.test * fix error in base case * fix data processor error and improve test to catch this kind of error
* add build-system like pr 1292 in fasttext repo * add pybind11 in build steps since broken with new pip tool * switch to fasttext-wheel to fix the build problem * update changelog
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
path_to_retrain_model
fasttext-wheel
instead offasttext
(see here and here).