Skip to content

Pass on idxs keyword argument from ensemble plots#493

Open
ChrisRackauckas wants to merge 3 commits into
masterfrom
ChrisRackauckas-patch-1
Open

Pass on idxs keyword argument from ensemble plots#493
ChrisRackauckas wants to merge 3 commits into
masterfrom
ChrisRackauckas-patch-1

Conversation

@ChrisRackauckas

Copy link
Copy Markdown
Member

No description provided.

Comment thread src/ensemble/ensemble_solutions.jl Outdated
Comment thread src/ensemble/ensemble_solutions.jl Outdated
ChrisRackauckas and others added 2 commits September 14, 2023 08:06
Co-authored-by: Qingyu Qu <52615090+ErikQQY@users.noreply.github.com>
Co-authored-by: Qingyu Qu <52615090+ErikQQY@users.noreply.github.com>
@ErikQQY

ErikQQY commented Sep 14, 2023

Copy link
Copy Markdown
Member

While testing locally, setting idxs=3 or idxs=(3, 5) still display all of the components

testtest

@codecov

codecov Bot commented Sep 14, 2023

Copy link
Copy Markdown

Codecov Report

Merging #493 (bf3d605) into master (73eb334) will decrease coverage by 0.44%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #493      +/-   ##
==========================================
- Coverage   57.00%   56.56%   -0.44%     
==========================================
  Files          50       50              
  Lines        3703     3707       +4     
==========================================
- Hits         2111     2097      -14     
- Misses       1592     1610      +18     
Files Changed Coverage Δ
src/ensemble/ensemble_solutions.jl 43.36% <0.00%> (-1.60%) ⬇️

... and 5 files with indirect coverage changes

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants