Simplify namespace
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 68/100
- Issue type
- Refactor
- Clarity
- Mostly clear
- Activity status
- Quiet
- Tech stack
- python
- Domain
- backend-api-design
Research direction
Search the repository for create_dolfinx_function_with_point_data and create_dolfinx_function_with_cell_data, including their tests and documentation. Rename the public entry points to read_point_data and read_cell_data consistently, then run the existing test suite to confirm the new API works and the old names are no longer referenced.
Written by the indexing model from the issue text.
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 foam2dolfinx
- Dominant language
- Python
- Stars
- 8
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from festim-dev/foam2dolfinx
-
Difficulty 2/5 1-3 hours Newbie friendliness 57/100
festim-dev/foam2dolfinx#42 ·
-
INFO feature Open
Difficulty 5/5 Over a week Newbie friendliness 35/100
festim-dev/foam2dolfinx#37 ·
All issues in festim-dev/foam2dolfinx
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100