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

Rename “isosurface“ to “adHocMesh” in code #7243

Closed
frcroth opened this issue Jul 31, 2023 · 7 comments · Fixed by #7350
Closed

Rename “isosurface“ to “adHocMesh” in code #7243

frcroth opened this issue Jul 31, 2023 · 7 comments · Fixed by #7350

Comments

@frcroth
Copy link
Member

frcroth commented Jul 31, 2023

Ad-hoc meshing may be a better term, backend still uses "isosurface", which is a quite general term (MATLAB legacy?)

@fm3
Copy link
Member

fm3 commented Jul 31, 2023

Yes, I’d say we should rename isosurface to adHocMesh in our code. I think we already decided to hide the term isosurface from the users. Adapting the code is the logical next step.

( Any objections @philippotto @normanrz ? )

@philippotto
Copy link
Member

Yes, I’d say we should rename isosurface to adHocMesh in our code. I think we already decided to hide the term isosurface from the users. Adapting the code is the logical next step.

I agree 👍

@fm3 fm3 removed the discussion label Jul 31, 2023
@fm3 fm3 changed the title Unify naming ad-hoc mesh / isosurface Rename “isosurface“ to „adHocMesh“ in code Jul 31, 2023
@fm3 fm3 changed the title Rename “isosurface“ to „adHocMesh“ in code Rename “isosurface“ to “adHocMesh” in code Jul 31, 2023
@frcroth
Copy link
Member Author

frcroth commented Sep 5, 2023

Do it after #7244

@frcroth
Copy link
Member Author

frcroth commented Sep 19, 2023

What is the scope of this issue? I did the renaming in the backend, which was straight-forward, however looking at the front-end, isosurface seems to have a slightly different meaning, because there are PrecomputedIsosurfaces and AdHocIsosurfaces, so I'm not sure what to do here.

@fm3
Copy link
Member

fm3 commented Sep 19, 2023

@philippotto could you comment on that? My guess is that we can safely rename AdHocIsosurfaces to AdHocMesh – should we also rename PrecomputedIsosurfaces in the frontend to PrecomputedMeshes? Do you see any dangers/problems there?

@philippotto
Copy link
Member

What is the scope of this issue? I did the renaming in the backend, which was straight-forward, however looking at the front-end, isosurface seems to have a slightly different meaning, because there are PrecomputedIsosurfaces and AdHocIsosurfaces, so I'm not sure what to do here.

The front-end uses isosurface as a synonym to mesh. So, the necessary renaming would be isosurface -> mesh. Feel free to do the renaming in the front-end, too.

@philippotto
Copy link
Member

@philippotto could you comment on that? My guess is that we can safely rename AdHocIsosurfaces to AdHocMesh – should we also rename PrecomputedIsosurfaces in the frontend to PrecomputedMeshes? Do you see any dangers/problems there?

I only just saw your comment. I don't see any problems with that. Renaming to PrecomputedMeshes should be fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants