In example code in document change value of script_repr_suppress_defaults = to False #1159
Open
Coderambling wants to merge 1 commit into
Open
In example code in document change value of script_repr_suppress_defaults = to False #1159Coderambling wants to merge 1 commit into
Coderambling wants to merge 1 commit into
Conversation
…eblock The doc says: "If you want a record of the complete set of parameter values, including all defaults, you can enable that behavior:" n the code block that follows, script_repr_suppress_defaults is set to True. But in order to show the complete set of parameter values, suppression should be set to False. Also, the document should mention the default value for script_repr_suppress_defaults. It seems that by default it is set to True. In the API reference documentation, I was unable to find mention of script_repr_suppress_defaults . This seems to be a documentation gap. Should I file a separate issue for this?
Coderambling
changed the title
Coderambling
changed the title
Coderambling
added a commit
to Coderambling/param
that referenced
this pull request
Jul 4, 2026
…ress_defaults Added mention of suppress flag (or is it a global config variable?) to the script_repr docstring, for awareness, clarification and completeness. See also holoviz#1159 . By default, script_repr prints only parameter values that have changed from their defaults; for the complete set of parameter values, including all defaults, first run: param.parameterized.script_repr_suppress_defaults=False For more details on this, see: https://param.holoviz.org/en/docs/latest/user_guide/Serialization_and_Persistence.html#script-repr-limitations-and-workarounds
Coderambling
added a commit
to Coderambling/param
that referenced
this pull request
Jul 4, 2026
…press_defaults Added mention of suppress flag (or is it a global config variable?) to the docstring of script_repr docstring, for awareness, clarification and completeness. See also holoviz#1159 . By default, script_repr prints only parameter values that have changed from their defaults; for the complete set of parameter values, including all defaults, first run: param.parameterized.script_repr_suppress_defaults=False For more details on this, see: https://param.holoviz.org/en/docs/latest/user_guide/Serialization_and_Persistence.html#script-repr-limitations-and-workarounds <!-- Using AI? READ FIRST: https://holoviz.org/contribute.html#ai-readme --> ## Description <!-- Summarize the change and which issue is fixed. Be sure to include relevant motivation and context. Describe the tests run to verify these changes and how to reproduce them (copy-pastable minimal, reproducible example). Include visuals when possible (e.g. before/after screenshots). --> Fixes #{issue} ## AI Disclosure <!-- Delete this section if AI was not used, else specify the tool/model (e.g. Cursor + Sonnet 4.6, Claude Code + Opus 4.6, ChatGPT) and briefly describe how it was used. Failure to disclose AI usage may result in a ban. --> Tool & Model: Usage: - [ ] I have tested all AI-generated content in my PR. - [ ] I take responsibility for all AI-generated content in my PR. ## Checklist <!-- Remove the non relevant items. --> - [ ] Tests added and are passing - [ ] Added documentation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The doc says:
"If you want a record of the complete set of parameter values, including all defaults, you can enable that behavior:"
In the code block that follows, script_repr_suppress_defaults is set to True. But in order to show the complete set of parameter values, suppression should be set to False.
Also, the document should mention the default value for script_repr_suppress_defaults. It seems that by default it is set to True.
In the API reference documentation, I was unable to find mention of script_repr_suppress_defaults . This seems to be a documentation gap. Should I file a separate issue for this?
Description
Fixes #{issue}
AI Disclosure
Tool & Model:
Usage:
Checklist