Skip to content

fix: test and avoid memory leaks by checking references to virtual kernel#377

Merged
maartenbreddels merged 13 commits intomasterfrom
11-13-fix_test_and_avoid_memory_leaks_by_checking_references_to_virtual_kernel
Dec 17, 2024
Merged

fix: test and avoid memory leaks by checking references to virtual kernel#377
maartenbreddels merged 13 commits intomasterfrom
11-13-fix_test_and_avoid_memory_leaks_by_checking_references_to_virtual_kernel

Conversation

@maartenbreddels
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor Author

maartenbreddels commented Nov 13, 2023

@maartenbreddels maartenbreddels force-pushed the 11-13-fix_test_and_avoid_memory_leaks_by_checking_references_to_virtual_kernel branch 23 times, most recently from 945fb87 to 53e1d6d Compare November 16, 2023 10:46
@maartenbreddels maartenbreddels changed the base branch from master to 11-21-perf_ipyvue_widget_can_use_a_faster_less_generate_state_get November 22, 2023 11:41
@maartenbreddels maartenbreddels force-pushed the 11-13-fix_test_and_avoid_memory_leaks_by_checking_references_to_virtual_kernel branch from 53e1d6d to 906c529 Compare November 22, 2023 11:41
@maartenbreddels maartenbreddels force-pushed the 11-21-perf_ipyvue_widget_can_use_a_faster_less_generate_state_get branch from 098277a to 4e040ae Compare January 3, 2024 17:07
@maartenbreddels maartenbreddels force-pushed the 11-13-fix_test_and_avoid_memory_leaks_by_checking_references_to_virtual_kernel branch from 4a7e206 to 34a3873 Compare January 3, 2024 17:07
@maartenbreddels maartenbreddels force-pushed the 11-21-perf_ipyvue_widget_can_use_a_faster_less_generate_state_get branch from 4e040ae to 8871ca8 Compare January 12, 2024 13:02
@maartenbreddels maartenbreddels force-pushed the 11-13-fix_test_and_avoid_memory_leaks_by_checking_references_to_virtual_kernel branch from 34a3873 to 956fac4 Compare January 12, 2024 13:02
@maartenbreddels maartenbreddels force-pushed the 11-21-perf_ipyvue_widget_can_use_a_faster_less_generate_state_get branch from 8871ca8 to f7ef3f3 Compare February 9, 2024 17:33
@maartenbreddels maartenbreddels force-pushed the 11-13-fix_test_and_avoid_memory_leaks_by_checking_references_to_virtual_kernel branch from 956fac4 to 23e8262 Compare February 9, 2024 17:33
@maartenbreddels maartenbreddels force-pushed the 11-21-perf_ipyvue_widget_can_use_a_faster_less_generate_state_get branch from f7ef3f3 to 63de078 Compare February 16, 2024 19:55
@maartenbreddels maartenbreddels force-pushed the 11-13-fix_test_and_avoid_memory_leaks_by_checking_references_to_virtual_kernel branch from 23e8262 to c9365e6 Compare February 16, 2024 19:55
@maartenbreddels maartenbreddels force-pushed the 11-21-perf_ipyvue_widget_can_use_a_faster_less_generate_state_get branch from 63de078 to 63e2dd3 Compare February 23, 2024 16:32
@maartenbreddels maartenbreddels force-pushed the 11-13-fix_test_and_avoid_memory_leaks_by_checking_references_to_virtual_kernel branch from c9365e6 to f29ebcc Compare February 23, 2024 16:32
@maartenbreddels maartenbreddels force-pushed the 11-21-perf_ipyvue_widget_can_use_a_faster_less_generate_state_get branch from 63e2dd3 to 01dd332 Compare March 10, 2024 21:24
@maartenbreddels maartenbreddels force-pushed the 11-13-fix_test_and_avoid_memory_leaks_by_checking_references_to_virtual_kernel branch from f29ebcc to 3b65943 Compare March 10, 2024 21:24
@maartenbreddels maartenbreddels force-pushed the 11-21-perf_ipyvue_widget_can_use_a_faster_less_generate_state_get branch from 01dd332 to ccac5e9 Compare March 11, 2024 15:24
@maartenbreddels
Copy link
Copy Markdown
Contributor Author

I manually changed the branch we will merge this to, so it does not depend on #377 anymore. @iisakkirotko can you fix this,or should we just merge this as is? (squashed)

@iisakkirotko
Copy link
Copy Markdown
Collaborator

I changed the base branch to master in graphite, is there anything else that I should do? I also opened #936 to debug the CI issue related to this, so from that side we're clear to merge 👍

@maartenbreddels
Copy link
Copy Markdown
Contributor Author

Nice, thanks for opening that PR!

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