Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issues in .apply dynamic handling #4025

Merged
merged 1 commit into from Oct 4, 2019
Merged

Fix issues in .apply dynamic handling #4025

merged 1 commit into from Oct 4, 2019

Conversation

@philippjfr
Copy link
Member

philippjfr commented Oct 4, 2019

Two small fixes for .apply:

  1. .apply(dynamic=False) works on 0 kdim DynamicMap
  2. dynamic variable is not overwritten which caused errors when using .apply on nested datastructures
@philippjfr philippjfr added the bug label Oct 4, 2019
@philippjfr philippjfr merged commit 9afc5b4 into master Oct 4, 2019
3 of 4 checks passed
3 of 4 checks passed
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls First build on apply_dynamic_fix at 88.193%
Details
philippjfr added a commit that referenced this pull request Oct 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.