| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
There was a problem hiding this comment.
Looks nice so far; thanks a lot for handling this! t
Two questions regarding the scope :
Sorry, something went wrong.
|
yess working at it! good suggestions |
Sorry, something went wrong.
|
For the docs happy to coordinate and add this to my ongoing updates! |
Sorry, something went wrong.
|
Note: adding some smoke tests in CI, which I will probably move to a separate PR later to keep the discussions separated. But first want to test for compatible versioning. I will also update the docs once finished and coordinate with you @C-Achard & help with the new docs to avoid parallel work. |
Sorry, something went wrong.
|
@C-Achard, it's ready for review now. Let me know what you think about the CI changes: for now I added a simple smoke test directly in the intelligent testing workflow, which means we can avoid (downloading and) installing ffmpeg for every run, and also doesn't affect the concurrency with the full-matrix tests. If you think it should make use of the python-package workflow, like the other test lanes, I can see arguments for that as well. |
Sorry, something went wrong.
There was a problem hiding this comment.
Can we cherry-pick/adapt changes to the latest docs audit updates ?
Sorry, something went wrong.
|
Seems tensorflow-macos is starting to fail in CI, perhaps we should make this high priority |
Sorry, something went wrong.
There was a problem hiding this comment.
Note: this has already been replaced by #3280 and can be safely discarded I think
Sorry, something went wrong.
- revise tf extra - add tf-cu11, tf-cu12, tf-latest This brings in line the CI testing with the actual extras provided, simplifies the installation, and allows for targeted installs for different users.
Xet is Hugging Face’s new storage backend designed to replace the aging Git LFS (Large File Storage). While Git LFS treats every file as a single "blob," Xet breaks files into small "chunks." In restricted CI environments (like CircleCI, GCP, or Kaggle), the Xet "chunked" download often stalls. Because Xet opens many simultaneous connections to different data chunks, CI firewalls or NAT gateways sometimes flag this as suspicious activity or simply drop the connections when they hit a certain threshold.
|
Cleaned up this PR and fixed failing tests:
|
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
The current tensorflow installation via the optional extra tf is designed as a one-solution-for-all, instead of providing a tailored fit for multiple users with different machines. Also, there is divergence between the recommendations in the docs, CI and the current pyproject.toml. This PR implements a more diverse set of tensorflow installations, and aligns the CI so that these versions are actually tested. The documentation is also updated accordingly.
Addresses #3271
Optional dependencies
The new configuration is as follows:
CI
Documentation
The documentation changes are now moved to a separate PR #3317.