Skip to content

new getBlobEntriesVariables#996

Merged
dehann merged 1 commit intomasterfrom
23Q1/enh/getblobentvars
Mar 27, 2023
Merged

new getBlobEntriesVariables#996
dehann merged 1 commit intomasterfrom
23Q1/enh/getblobentvars

Conversation

@dehann
Copy link
Copy Markdown
Member

@dehann dehann commented Mar 27, 2023

convenience function for gathering many blob entries over variables

@dehann dehann self-assigned this Mar 27, 2023
@dehann dehann added enhancement New feature or request feature request labels Mar 27, 2023
@dehann dehann requested a review from Affie March 27, 2023 07:33
@dehann dehann added this to the v0.20.2 milestone Mar 27, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 27, 2023

Codecov Report

Merging #996 (c22a5dc) into master (a78b6d9) will decrease coverage by 0.39%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #996      +/-   ##
==========================================
- Coverage   83.46%   83.07%   -0.39%     
==========================================
  Files          25       25              
  Lines        1947     1956       +9     
==========================================
  Hits         1625     1625              
- Misses        322      331       +9     
Impacted Files Coverage Δ
src/DataBlobs/services/BlobEntry.jl 60.76% <0.00%> (-4.53%) ⬇️
src/DataBlobs/services/HelpersDataWrapEntryBlob.jl 77.08% <ø> (ø)
src/DistributedFactorGraphs.jl 100.00% <ø> (ø)

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

Copy link
Copy Markdown
Member

@Affie Affie 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, we might just consider having the variable label in the results, but I don't know if your use case needs it.

@dehann
Copy link
Copy Markdown
Member Author

dehann commented Mar 27, 2023

thanks, thought about that a bit and think the varList keyword and dropEmpties gives enough freedom if user wants to keep association of variable to blobs. case i need it for is just get all the blobs matching the pattern

@dehann dehann merged commit e79bed1 into master Mar 27, 2023
@dehann dehann deleted the 23Q1/enh/getblobentvars branch March 27, 2023 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants