Skip to content

65 address shared memory issues #67

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.

Sign up for GitHub

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

Jump to bottom
Merged
den-sq merged 6 commits into from
Dec 30, 2025
Merged

65 address shared memory issues #67

den-sq merged 6 commits into from
Dec 30, 2025

Conversation

@den-sq
Copy link
Collaborator

@den-sq den-sq commented Dec 30, 2025

Closes #65

  • Updated shared memory manager to handle shorter-term memory blocks
  • Clears up memory block as they are resolved instead of waiting.
  • Refactoring slice pipeline to handle this properly and use fetch / slicing processes.

den-sq and others added 6 commits December 30, 2025 15:18
@den-sq
Merge pull request #66 from ChengLabResearch/main
a3a7468 
Increase shm_size for docker to avoid memory issue.
@den-sq
Update Shared Memory Manager to allow early deletion
66687b2 
- Termed np arrays can be deleted before manager is shutdown.
- Otherwise function similarly to existing.
@den-sq
Update Slicing Pipeline SHM handling
830a8d8 
- Downloading now a process pipeline
- Shared Memory properly destroyed early (due to use of termed arrays)
- Process split set as 1:3 DL:Process
@den-sq
Merge branch '65-address-shared-memory-issues' of https://github.com/…
e577bcd 
…ChengLabResearch/ouroboros into 65-address-shared-memory-issues
@den-sq
- request_volume_for_slice update for separate download call
dcf202b 
- test updates
@github-actions
Updated coverage.svg
f84eac8
@den-sq den-sq linked an issue Dec 30, 2025 that may be closed by this pull request
Address Shared Memory Issues #65
Closed
@den-sq den-sq merged commit 973729a into main Dec 30, 2025
1 check passed
@den-sq den-sq deleted the 65-address-shared-memory-issues branch December 30, 2025 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Address Shared Memory Issues

2 participants

@den-sq