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

Heat exchanger example seems odd #79

Closed
flothesof opened this issue Mar 14, 2019 · 3 comments
Closed

Heat exchanger example seems odd #79

flothesof opened this issue Mar 14, 2019 · 3 comments

Comments

@flothesof
Copy link

Hi there,

thank you for putting together the nice new looking documentation.
I'm trying to get started with FreeFem and read through the heat exchanger example.
I just wanted to say two things about this example:

  • I first got confused about the modelling of this situation: one conductor is modelled as a boundary condition (C1 at 60°C) while the other is as a volume.
  • the text says that kappa in conductor C2 is 5 times larger than in C0 but the code says 3 times larger Vh kappa=1 + 2*(x<-1)*(x>-2)*(y<3)*(y>-3);

I think that this example could be made more beginner friendly by clearly putting an illustration of the situation with C0, C1 and C2 at the beginning of the problem description. Otherwise, adjusting the description to the code would reduce confusion (Vh kappa=1 + 4*(x<-1)*(x>-2)*(y<3)*(y>-3);).

Best regards,
Florian

@sgarnotel
Copy link
Member

Hi,

Tanks a lot for your feedback!

This is designed as is because we have a transmitter (C1) and a receiver (C2), the first one volume (surface in 2D) is not important, so not meshed.

I will fix the value of kappa and add an illustration of the problem.

@sgarnotel
Copy link
Member

Fixed

If you have any remark, don't hesitate

@flothesof
Copy link
Author

Thanks for your quick action. I just checked the illustration you added, it looks great in the tutorial! Great work :)

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