8000 Small bug fixes by hoxbro · Pull Request #4481 · holoviz/panel · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@hoxbro
Copy link
Member
@hoxbro hoxbro commented Mar 1, 2023

This fixes two things:

  • Make nan=nan comparison to True, as I don't see a reason why it shouldn't be for our case.
  • Use cache from previous sessions when using to_disk

@hoxbro hoxbro force-pushed the small_nan_compare branch from 13e85df to f1026f0 Compare March 1, 2023 13:00
@codecov
Copy link
codecov bot commented Mar 1, 2023

Codecov Report

Merging #4481 (f1026f0) into main (6053d67) will decrease coverage by 8.14%.
The diff coverage is 50.00%.

❗ Current head f1026f0 differs from pull request most recent head 8b4326a. Consider uploading reports for the commit 8b4326a to get more accurate results

@@            Coverage Diff             @@
##             main    #4481      +/-   ##
==========================================
- Coverage   82.22%   74.09%   -8.14%     
==========================================
  Files         243      243              
  Lines       35320    35319       -1     
==========================================
- Hits        29043    26170    -2873     
- Misses       6277     9149    +2872     
Flag Coverage Δ
ui-tests ?
unitexamples-tests 74.09% <50.00%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
panel/io/model.py 61.11% <0.00%> (ø)
panel/tests/util.py 81.98% <ø> (-8.11%) ⬇️
panel/io/cache.py 85.34% <100.00%> (-0.07%) ⬇️
panel/tests/ui/widgets/test_tabulator.py 9.45% <0.00%> (-88.77%) ⬇️
panel/tests/ui/widgets/test_input.py 11.34% <0.00%> (-88.13%) ⬇️
panel/tests/ui/layout/test_accordion.py 14.72% <0.00%> (-85.28%) ⬇️
panel/tests/ui/widgets/test_sliders.py 15.42% <0.00%> (-84.58%) ⬇️
panel/tests/ui/layout/test_card.py 18.48% <0.00%> (-81.52%) ⬇️
panel/tests/ui/pane/test_image.py 25.80% <0.00%> (-74.20%) ⬇️
panel/tests/ui/test_reactive.py 26.53% <0.00%> (-73.47%) ⬇️
... and 28 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member
@philippjfr philippjfr left a comment

Choose a reason for hiding this comment

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

Looks good!

8000
@philippjfr philippjfr merged commit f4caa1e into main Mar 1, 2023
@philippjfr philippjfr deleted the small_nan_compare branch March 1, 2023 17:47
maximlt pushed a commit that referenced this pull request Mar 2, 2023
* comparable_array with nan_equal=True

* Fix cache
maximlt added a commit that referenced this pull request Mar 3, 2023
* keywords and like filters aren't regex (#4423)
* add examples folder dynamically added to the distrib (#4484)
* Small bug fixes  (#4481)
* use the right name for the method that gets the image data
* Bump cryptography in /examples/apps/django_multi_apps (#4418)
* Use latest react-grid from CDN (#4461)
* Fix url
* Fix React template
* prepare release notes
* Modify sys.path when running inside Jupyter Kernel (#4489)
* Add support for altair and vega-lite v5 (#4488)
* Do not re-create Vega.selections object unless selections changed (#4497)
* Bump panel.js version to 0.14.4-rc.2

---------

Co-authored-by: Simon Høxbro Hansen <simon.hansen@me.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Philipp Rudiger <prudiger@anaconda.com>
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0