Skip to content

Allow to disable writing computation times#1385

Merged
larsoner merged 3 commits intosphinx-gallery:masterfrom
bmwiedemann:benchmark
Oct 7, 2024
Merged

Allow to disable writing computation times#1385
larsoner merged 3 commits intosphinx-gallery:masterfrom
bmwiedemann:benchmark

Conversation

@bmwiedemann
Copy link
Copy Markdown

Allow to disable writing computation times to allow for reproducible builds.

Default to not write durations. if SOURCE_DATE_EPOCH indicates that a reproducible build is wanted.

Helps with pygraphviz/pygraphviz#541

@larsoner
Copy link
Copy Markdown
Contributor

larsoner commented Oct 4, 2024

Makes sense to me, can you add it to the configuration doc somewhere?

to allow for reproducible builds.

Default to not write durations. if SOURCE_DATE_EPOCH indicates
that a reproducible build is wanted.

Helps with pygraphviz/pygraphviz#541

See https://reproducible-builds.org/ for why this matters.
@bmwiedemann
Copy link
Copy Markdown
Author

I added some docs. Feel free to adjust it as you like.

Write computation times
=======================

Set to ``false`` if you want to omit computation times from the output.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
Set to ``false`` if you want to omit computation times from the output.
Set to ``False`` if you want to omit computation times from the output.

Copy link
Copy Markdown
Contributor

@lucyleeow lucyleeow left a comment

Choose a reason for hiding this comment

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

LGTM

@larsoner larsoner enabled auto-merge (squash) October 7, 2024 15:31
@larsoner larsoner disabled auto-merge October 7, 2024 15:31
@larsoner larsoner enabled auto-merge (squash) October 7, 2024 15:31
@larsoner larsoner merged commit d0fed09 into sphinx-gallery:master Oct 7, 2024
bmwiedemann added a commit to bmwiedemann/pygraphviz that referenced this pull request Oct 14, 2024
MridulS pushed a commit to pygraphviz/pygraphviz that referenced this pull request Oct 17, 2024
@bmwiedemann bmwiedemann deleted the benchmark branch June 5, 2025 06:13
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