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

Polish language on IBAMR section #10

Open
labarba opened this issue Aug 8, 2016 · 1 comment
Open

Polish language on IBAMR section #10

labarba opened this issue Aug 8, 2016 · 1 comment

Comments

@labarba
Copy link
Member

labarba commented Aug 8, 2016

Some of the language in the section reporting our experience with IBAMR can be read as direct criticism towards the team that developed this library. We've heard from the IBAMR team that they object to our description.

In particular, the sentence "The published record is incomplete in this regard: we could find no explanation for it [the use of a solid body discretization instead of a boundary discretization] in any paper using IBAMR" was interpreted as an accusation that the papers deliberately leave out important details. This was not our intention in writing the text. We simply want to report our experience trying to learn how to use this code, and our mistakes in the process.

We also need to correct the description of the outflow boundary conditions, which are not zero-gradient conditions, but traction-free conditions. This was our mistake in the description. And as the discussion following the post in our group web site shows, the IBAMR team is worried that readers could misinterpret Figure 5 as "a fundamental problem with the library." We need to edit the figure caption to make clear that it was our error setting the boundary condition, at first, and that we corrected it later. The section does say we successfully replicated previous results with IBAMR, but it can be said also in the caption.

@mesnardo
Copy link
Member

mesnardo commented Aug 18, 2016

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

No branches or pull requests

2 participants