Skip to content
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.

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

Shared Memory functionality #69

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
Open

Shared Memory functionality #69

wants to merge 22 commits into from

Conversation

Vogull
Copy link
Collaborator

@Vogull Vogull commented Mar 25, 2024

Added shared memory functionality for input and output of Evaluate, Gradient, Jacobian and Hessian functions. Automatic fallback to http/json if test transmission fails.

Discussing possible improvements.

linusseelinger and others added 7 commits November 14, 2023 10:50
Shared memory functionality is expanded and now supports input and output for: Evaluate, Gradient, Jacobian and Hessian.

Umbridge automatically attempts a shared memory transmission in order to detect accessibility and falls back to full HTTP/JSON transmission on failure.

Currently, the shared memory segments will only be compiled on Linux, may be expanded to Apple Mac (needs testing).
Shared memory functionality is expanded and now supports input and output for: Evaluate, Gradient, Jacobian and Hessian.

Umbridge automatically attempts a shared memory transmission in order to detect accessibility and falls back to full HTTP/JSON transmission on failure.

Works on Linux and Windows. MacOS is to be tested.
Fix indentation for comparison
@linusseelinger
Copy link
Member

Looks good, thanks a lot!

Can you remove the draft status? Apparently there's also a potential merge conflict, could you look into that?

@Vogull Vogull marked this pull request as ready for review April 21, 2024 22:08
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.

None yet

2 participants