Skip to content

Conversation

@shadeMe
Copy link
Contributor

@shadeMe shadeMe commented Sep 9, 2022

Sync branches to fix GPU tests (#757).

rmitsch and others added 9 commits August 26, 2022 16:23
* Remove redundant tests. Add confection to requirement.txt and setup.cfg. Adjust cnfig.py.

* Add reference to confection in website/docs/usage-config.md.

* Update confection reference in docs.

* Extend imports fro confection for backwards compatibility.
* `PyTorchGradScaler`: Cache `_found_inf` on the CPU

This prevents unnecessary overhead from launching kernels on the GPU in hot backward passes.

* Only pin `_found_inf` to the CPU

* Always store `_found_inf` as a `bool`
* work with cupy arrays and 2d arrays

* force mypy pass

* addressing comments

* return correct shape empty array

* test remap_ids with Ints2d

* Update thinc/layers/remap_ids.py

Co-authored-by: Daniël de Kok <[email protected]>

* use numpy array

* remove cupy import

* mini fix

* more strict typing

* adjust test

* Update thinc/layers/remap_ids.py

Co-authored-by: Adriane Boyd <[email protected]>

* remove check

* Update thinc/layers/remap_ids.py

Co-authored-by: Adriane Boyd <[email protected]>

* address reviews

* Update thinc/layers/remap_ids.py

Co-authored-by: Adriane Boyd <[email protected]>

* simplify casting

* Update thinc/layers/remap_ids.py

Co-authored-by: Adriane Boyd <[email protected]>

* Update thinc/layers/remap_ids.py

Co-authored-by: Adriane Boyd <[email protected]>

* remap_ids legacy

* legacy

* test version 1 and 2

* rename legacy to v1

* adding old test back

* remap_ids docs update

* Update website/docs/api-layers.md

Co-authored-by: Adriane Boyd <[email protected]>

* Update website/docs/api-layers.md

Co-authored-by: Adriane Boyd <[email protected]>

* make init/forward attribute setting more clear

* Update website/docs/api-layers.md

Co-authored-by: Adriane Boyd <[email protected]>

* Update website/docs/api-layers.md

Co-authored-by: Adriane Boyd <[email protected]>

* Update website/docs/api-layers.md

Co-authored-by: Adriane Boyd <[email protected]>

* prettier

* update model type

* prettier

* Use new _v2 instead of renamed _v1

Co-authored-by: Daniël de Kok <[email protected]>
Co-authored-by: Adriane Boyd <[email protected]>
…w` (explosion#757)

`tf.experimental.dlpack.from_dlpack` expects a `PyCapsule` object.
* Remove references to FastAPI being an Explosion product.

* Remove period at end of subheader.
@shadeMe shadeMe added the 🔜 v9.0 Related to upcoming v9.0 label Sep 9, 2022
@svlandeg svlandeg merged commit a80d275 into explosion:v9 Sep 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔜 v9.0 Related to upcoming v9.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants