festim-dev / festim-dev/foam2dolfinx

Simplify namespace

Open
#41 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
8
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Instead of `create_dolfinx_function_with_point_data(...)` and `create_dolfinx_function_with_cell_data` I think it would be a lot simpler to have `.read_point_data()` and `.read_cell_data()` which is the convention `io4dolfinx` uses.

"create dolfinx function` is a bit redundant since we already know this is foam2**dolfinx**

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.