Skip to content

Conversation

@siliataider
Copy link
Contributor

@siliataider siliataider commented Jun 18, 2025

This Pull request:

Changes or fixes:

JSROOT is on by default when opening a new notebook, users may want to disable it for their sessions without having to add one line per notebook, but instead one line in .rootrc that applies for all.

This PR adds the functionality to disable JSROOT in Jupyter notebooks by default through a new .rootrc parameter JSROOT.Disable.

Checklist:

This PR fixes #19068

Copy link
Member

@linev linev left a comment

Choose a reason for hiding this comment

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

Very nice.
But please change variable name

@siliataider siliataider requested a review from linev June 18, 2025 11:35
Copy link
Member

@linev linev left a comment

Choose a reason for hiding this comment

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

Thanks!

@siliataider siliataider force-pushed the jsroot branch 2 times, most recently from 2a11777 to 689e741 Compare June 18, 2025 11:46
Copy link
Member

@dpiparo dpiparo left a comment

Choose a reason for hiding this comment

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

Thanks for these changes!

@github-actions
Copy link

github-actions bot commented Jun 18, 2025

Test Results

    19 files      19 suites   3d 13h 12m 54s ⏱️
 2 856 tests  2 853 ✅ 0 💤 3 ❌
52 755 runs  52 752 ✅ 0 💤 3 ❌

For more details on these failures, see this check.

Results for commit d6ef1d7.

♻️ This comment has been updated with latest results.

@siliataider siliataider merged commit 59b7f40 into root-project:master Jun 19, 2025
41 of 44 checks passed
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.

[jupyter] let disable jsroot by default

3 participants