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

Making BlackOilFluidState GPU friendly #4472

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kjetilly
Copy link
Contributor

This is in draft at the moment. Essentially this adds decorators to the functions and eventually will add support non-static fluidsystem-objects.

@bska
Copy link
Member

bska commented Feb 10, 2025

Is there any relation between this work and PR #4427?

@kjetilly
Copy link
Contributor Author

Is there any relation between this work and PR #4427?

Yes, in the sense that the BlackOilFluidState object will eventually use the non-static variant of the BlackOilFluidSystem developed in that pull-request, but this PR will work without 4427 (or at least, that is the goal for now).

@kjetilly kjetilly force-pushed the blackoilfluidstategpu branch from 848a3ec to f4ff4f3 Compare February 11, 2025 12:11
@kjetilly
Copy link
Contributor Author

Jenkins build this opm-simulators=5971 please

1 similar comment
@kjetilly
Copy link
Contributor Author

Jenkins build this opm-simulators=5971 please

@kjetilly kjetilly force-pushed the blackoilfluidstategpu branch from ed1596d to fcba8e7 Compare February 12, 2025 09:45
@kjetilly kjetilly force-pushed the blackoilfluidstategpu branch from fcba8e7 to 3f71dd0 Compare February 24, 2025 09:26
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