+
Skip to content

Merge dev to main for v1.1.1 #1452

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

Merged
merged 31 commits into from
Apr 9, 2024
Merged

Merge dev to main for v1.1.1 #1452

merged 31 commits into from
Apr 9, 2024

Conversation

bdu91
Copy link
Collaborator

@bdu91 bdu91 commented Apr 9, 2024

No description provided.

iftenney and others added 30 commits February 21, 2024 12:48
PiperOrigin-RevId: 609090874
PiperOrigin-RevId: 609143233
PiperOrigin-RevId: 610789991
PiperOrigin-RevId: 610927735
PiperOrigin-RevId: 611617505
…n preparation for generalization to more models.

PiperOrigin-RevId: 615558614
PiperOrigin-RevId: 615594853
Text color changing was a bit jarring, so set default range to half of the color
scale. Intensity slider now changes the scale range as well as gamma.

PiperOrigin-RevId: 615821669
- <lit-text-chips> as version of <lit-token-chips> for running text.
- Clean up some redundant CSS names and rules in token_chips.css.

PiperOrigin-RevId: 616166852
Toggle icon to switch between this and the regular background highlight.

PiperOrigin-RevId: 616175592
…mple comparison.

With this version, only the selected datapoint will be editable, not the pinned one.

PiperOrigin-RevId: 616246154
This doesn't rename the classes; will do that in a follow-up.

PiperOrigin-RevId: 616256449
PiperOrigin-RevId: 616272334
Left-side toolbar controls were one long non-wrapping block, so if the module
was narrow enough these would clip. This splits them into two blocks, allowing
them to wrap, and adds a spacer to maintain the intended positioning.

PiperOrigin-RevId: 616319869
Instead of matching segment prefixes, this treats the full match as it's own
segment. Use this instead of ugly lookbehind assertions for line and paragraph
splitting.

PiperOrigin-RevId: 618012304
PiperOrigin-RevId: 620880580
Summary of changes:

* Wherever possible, use `keras.ops` for Tensor operations. These functions are designed to create and operate over Tensors from any of the Keras 3 backends.
* Identified a common API surface for salience predictions that supports all existing (and likely any future) Keras 3 backends. Functions implementing this API shoudl be named `_pred_{framework}` and return a tuple of the GradNorm and GradDotInput salience scores.
* Refactors `KerasSalienceModel._pred()` to perform common operations for data before calling out to backend-specific prediction functions.
* Extracts TensorFlow code in `KerasSalienceModel` to a new `_pred_tf()` function.
* Implements a `KerasSalienceModel._pred_torch()` function based on the HF implementation in lit_nlp/examples/models/pretrained_lms.py
* Provides a stub for `KerasSalienceModel._pred_jax()` with a detailed comment outlining the JAX idiosyncrasies that we need to adapt to in order to support this backend.

PiperOrigin-RevId: 622966761
PiperOrigin-RevId: 622999169
Copy link

google-cla bot commented Apr 9, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@bdu91 bdu91 requested review from RyanMullins and iftenney April 9, 2024 22:28
@bdu91 bdu91 self-assigned this Apr 9, 2024
@bdu91 bdu91 merged commit 72e8479 into main Apr 9, 2024
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.

3 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载