Skip to content

[FIX] Group by: Restore aggregations if removed due to open context#5823

Merged
janezd merged 1 commit into
biolab:masterfrom
VesnaT:groupby_restore_agg
Feb 4, 2022
Merged

[FIX] Group by: Restore aggregations if removed due to open context#5823
janezd merged 1 commit into
biolab:masterfrom
VesnaT:groupby_restore_agg

Conversation

@VesnaT
Copy link
Copy Markdown
Contributor

@VesnaT VesnaT commented Feb 4, 2022

Issue

Fixes #5802

Description of changes
Includes
  • Code changes
  • Tests
  • Documentation

@VesnaT VesnaT changed the title [FIX] Group by: Restore aggregations if removed due to open context [WIP] Group by: Restore aggregations if removed due to open context Feb 4, 2022
@VesnaT VesnaT changed the title [WIP] Group by: Restore aggregations if removed due to open context [FIX] Group by: Restore aggregations if removed due to open context Feb 4, 2022
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 4, 2022

Codecov Report

Merging #5823 (9212a76) into master (d6c3792) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #5823      +/-   ##
==========================================
- Coverage   86.14%   86.14%   -0.01%     
==========================================
  Files         316      316              
  Lines       66421    66423       +2     
==========================================
+ Hits        57216    57217       +1     
- Misses       9205     9206       +1     

@janezd janezd merged commit 3516fc3 into biolab:master Feb 4, 2022
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.

Group By: KeyError with newly added Variables

2 participants