Skip to content

python310Packages.bokeh: Build from source & enable tests#215406

Closed
mweinelt wants to merge 4 commits intoNixOS:masterfrom
mweinelt:bokeh-fixup
Closed

python310Packages.bokeh: Build from source & enable tests#215406
mweinelt wants to merge 4 commits intoNixOS:masterfrom
mweinelt:bokeh-fixup

Conversation

@mweinelt
Copy link
Copy Markdown
Member

@mweinelt mweinelt commented Feb 9, 2023

To enable the tests we have to fetch the sources from Git, which in turn requires us to build the static files from source.

Bokeh uses setuptools-git-versioning, which has no version override, so we also fix the broken fallback version in the package metadata.

Still some failing tests towards and each iteration takes 10-12 minutes and it seems like at some point the selenium stuff just breaks down.

Description of changes
Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.05 Release Notes (or backporting 22.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@github-actions github-actions bot added the 6.topic: python Python is a high-level, general-purpose programming language. label Feb 9, 2023
@dotlambda
Copy link
Copy Markdown
Member

Something similar would be nice for python3Packages.panel.

@ofborg ofborg bot added 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Feb 9, 2023
@mweinelt
Copy link
Copy Markdown
Member Author

mweinelt commented Feb 9, 2023

Something similar would be nice for python3Packages.panel.

No doubt about that. These changes did cost me some time and energy already, so it's not in the cards for now.

@dotlambda
Copy link
Copy Markdown
Member

dotlambda commented Feb 9, 2023

Something similar would be nice for python3Packages.panel.

No doubt about that. These changes did cost me some time and energy already, so it's not in the cards for now.

Yeah, this looks like a lot of work. I'd hope that panel is set up somewhat similarly but I totally understand that you don't feel like tackling that now.

@mweinelt mweinelt marked this pull request as ready for review February 9, 2023 02:30
@mweinelt mweinelt force-pushed the bokeh-fixup branch 2 times, most recently from b489ae3 to 400ad68 Compare February 9, 2023 02:34
@dotlambda
Copy link
Copy Markdown
Member

Result of nixpkgs-review pr 215406 run on x86_64-darwin 1

3 packages marked as broken and skipped:
  • python310Packages.dalle-mini
  • python310Packages.optuna
  • python310Packages.rl-coach
16 packages failed to build:
  • python310Packages.datashader
  • python310Packages.holoviews
  • python310Packages.hvplot
  • python310Packages.livelossplot
  • python310Packages.nbsmoke
  • python310Packages.panel
  • python310Packages.spacy
  • python310Packages.spacy-loggers
  • python310Packages.spacy-transformers
  • python310Packages.textacy
  • python310Packages.wandb
  • python311Packages.holoviews
  • python311Packages.hvplot
  • python311Packages.livelossplot
  • python311Packages.nbsmoke
  • python311Packages.panel
2 packages built:
  • python310Packages.bokeh
  • python311Packages.bokeh

@dotlambda
Copy link
Copy Markdown
Member

Result of nixpkgs-review pr 215406 run on aarch64-linux 1

7 packages marked as broken and skipped:
  • python310Packages.dalle-mini
  • python310Packages.datashader
  • python310Packages.optuna
  • python310Packages.rl-coach
  • python310Packages.spacy
  • python310Packages.spacy-transformers
  • python310Packages.textacy
10 packages failed to build:
  • python310Packages.holoviews
  • python310Packages.hvplot
  • python310Packages.livelossplot
  • python310Packages.nbsmoke
  • python310Packages.panel
  • python311Packages.holoviews
  • python311Packages.hvplot
  • python311Packages.livelossplot
  • python311Packages.nbsmoke
  • python311Packages.panel
4 packages built:
  • python310Packages.bokeh
  • python310Packages.spacy-loggers
  • python310Packages.wandb
  • python311Packages.bokeh

@dotlambda
Copy link
Copy Markdown
Member

Result of nixpkgs-review pr 215406 run on aarch64-darwin 1

7 packages marked as broken and skipped:
  • python310Packages.dalle-mini
  • python310Packages.datashader
  • python310Packages.optuna
  • python310Packages.rl-coach
  • python310Packages.spacy
  • python310Packages.spacy-transformers
  • python310Packages.textacy
12 packages failed to build:
  • python310Packages.holoviews
  • python310Packages.hvplot
  • python310Packages.livelossplot
  • python310Packages.nbsmoke
  • python310Packages.panel
  • python310Packages.spacy-loggers
  • python310Packages.wandb
  • python311Packages.holoviews
  • python311Packages.hvplot
  • python311Packages.livelossplot
  • python311Packages.nbsmoke
  • python311Packages.panel
2 packages built:
  • python310Packages.bokeh
  • python311Packages.bokeh

@dotlambda
Copy link
Copy Markdown
Member

Result of nixpkgs-review pr 215406 run on x86_64-linux 1

3 packages marked as broken and skipped:
  • python310Packages.dalle-mini
  • python310Packages.optuna
  • python310Packages.rl-coach
11 packages failed to build:
  • python310Packages.datashader
  • python310Packages.holoviews
  • python310Packages.hvplot
  • python310Packages.livelossplot
  • python310Packages.nbsmoke
  • python310Packages.panel
  • python311Packages.holoviews
  • python311Packages.hvplot
  • python311Packages.livelossplot
  • python311Packages.nbsmoke
  • python311Packages.panel
7 packages built:
  • python310Packages.bokeh
  • python310Packages.spacy
  • python310Packages.spacy-loggers
  • python310Packages.spacy-transformers
  • python310Packages.textacy
  • python310Packages.wandb
  • python311Packages.bokeh

@mweinelt
Copy link
Copy Markdown
Member Author

mweinelt commented Feb 9, 2023

holoviz/panel#4097

Panel isn't compatible with bokeh 3.0 yet.

@mweinelt mweinelt marked this pull request as draft February 9, 2023 13:00
@ofborg ofborg bot requested a review from costrouc February 9, 2023 13:38
@mweinelt
Copy link
Copy Markdown
Member Author

Panel 1.0.0 requries bokeh 3.1.1. Rebased and debugg bokeh build and working on panel build.

@mweinelt mweinelt force-pushed the bokeh-fixup branch 3 times, most recently from 431ecbe to e6a40a5 Compare May 25, 2023 10:03
@mweinelt mweinelt force-pushed the bokeh-fixup branch 2 times, most recently from 3426245 to 85a607e Compare June 5, 2023 21:58
@mweinelt mweinelt force-pushed the bokeh-fixup branch 2 times, most recently from 0a94699 to 8f2bac0 Compare June 22, 2023 11:08
mweinelt added 4 commits July 25, 2023 18:49
https://docs.bokeh.org/en/latest/docs/releases.html#release-3-2-0

Changes the build to a source build and enables tests.

To enable the tests we have to fetch the sources from Git, which in turn
requires us to build the static files from source.

Bokeh uses setuptools-git-versioning, which has no version override, so
we also fix the broken fallback version in the package metadata.

We skip sampledata tests, because that would require an additional
download, and we skip selenium tests because they are crashing with
chromium/chromedriver and getting out of sync with geckodriver/firefox.
python310Packages.bokeh: 2.4.3 -> 3.2.1

https://docs.bokeh.org/en/latest/docs/releases.html#release-3-2-1

Changes the build to a source build and enables tests.

To enable the tests we have to fetch the sources from Git, which in turn
requires us to build the static files from source.

Bokeh uses setuptools-git-versioning, which has no version override, so
we also fix the broken fallback version in the package metadata.

We skip sampledata tests, because that would require an additional
download, and we skip selenium tests because they are crashing with
chromium/chromedriver and getting out of sync with geckodriver/firefox.
@wegank wegank added 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md 2.status: merge conflict This PR has merge conflicts with the target branch labels Mar 19, 2024
@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Mar 20, 2024
@wegank wegank added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jul 4, 2024
@NickCao
Copy link
Copy Markdown
Member

NickCao commented Dec 4, 2024

Tests are enabled in #252993

@NickCao NickCao closed this Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge conflict This PR has merge conflicts with the target branch 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md 6.topic: python Python is a high-level, general-purpose programming language. 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants