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

Update event schema version type #421

Merged
merged 4 commits into from
Dec 18, 2024

Conversation

davidbrochart
Copy link
Collaborator

No description provided.

Copy link
Contributor

Binder 👈 Launch a Binder on branch davidbrochart/jupyter_collaboration/event-version

@davidbrochart
Copy link
Collaborator Author

It looks like UI tests are flaky again @trungleduc?
For instance:

  • actual:
    image
  • expected:
    image
  • diff:
    image

@trungleduc
Copy link
Member

yeah, your only choice is to relax the diff ratio

@@ -20,7 +20,7 @@ module.exports = {
},
expect: {
toMatchSnapshot: {
maxDiffPixelRatio: 0.0001
maxDiffPixelRatio: 0.01
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
maxDiffPixelRatio: 0.01
maxDiffPixelRatio: 0.001

1% might be overkill

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was not, I'm reverting that change.

@davidbrochart
Copy link
Collaborator Author

Ubuntu test failures are tracked in #416.

@krassowski
Copy link
Member

Ubuntu test failures are tracked in #416.

Should be fixed in 15 minutes with release of 4.3.4 🤞

@davidbrochart
Copy link
Collaborator Author

Oh, let's wait for it then 👍

@davidbrochart
Copy link
Collaborator Author

Let's see if it picks up JupyterLab v4.3.4.

@davidbrochart davidbrochart reopened this Dec 18, 2024
@davidbrochart
Copy link
Collaborator Author

CI is green again 🎉

@davidbrochart davidbrochart merged commit 0a75715 into jupyterlab:main Dec 18, 2024
47 of 52 checks passed
@davidbrochart davidbrochart deleted the event-version branch December 18, 2024 15:45
andrii-i pushed a commit to andrii-i/jupyter-collaboration that referenced this pull request Feb 11, 2025
* Update event schema version type

* Tweak maxDiffPixelRatio

* Set maxDiffPixelRatio to 0.001

Co-authored-by: Duc Trung Le <[email protected]>

* Set maxDiffPixelRatio to 0.01

---------

Co-authored-by: Duc Trung Le <[email protected]>
andrii-i pushed a commit to andrii-i/jupyter-collaboration that referenced this pull request Feb 11, 2025
* Update event schema version type

* Tweak maxDiffPixelRatio

* Set maxDiffPixelRatio to 0.001

Co-authored-by: Duc Trung Le <[email protected]>

* Set maxDiffPixelRatio to 0.01

---------

Co-authored-by: Duc Trung Le <[email protected]>
@brichet
Copy link
Contributor

brichet commented Feb 12, 2025

@meeseeksdev please backport to 3.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyter-collaboration that referenced this pull request Feb 12, 2025
davidbrochart added a commit that referenced this pull request Feb 12, 2025
* Backport PR #421: Update event schema version type

* Bump jupyter_server to >=2.15.0<3.0.0

---------

Co-authored-by: David Brochart <[email protected]>
Co-authored-by: Nicolas Brichet <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants