-
Notifications
You must be signed in to change notification settings - Fork 0
Add tracking of c/n in cake #34
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
Conversation
f16ef09 to
b0846a9
Compare
|
Tests have been updated and additionally |
schmoelder
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's move the formatting commit to its own PR.
|
Thanks for taking care of this @daklauss
Is there a technical reason for this? I believe, the equations should be the same as for |
b0846a9 to
4009cc5
Compare
|
They are the same but even the permeate tank is crashing with In hindsight im a little bit confused why |
Hmm, they should not be the same. |
No particles are leaving the cake, so every particle streaming into the cake is part of the cake? It would be more of an issue to use different |
|
I don't think so. If n_in = 0, that does not mean that n_cake will probably become 0. It just won't change. |
|
So So i guess in our case they are the same. I mean the only difference would be if you would to set the filtercake sectionwise to 0, then |
Yes, they are the same since it's assumed an ideal stirred tank.
I's suggest to track them separately. While they are numerically the same, we want to differentiate the unit operation outlet (which is a general interface which should be consistent across all unit operations) and the internal state. How is this done in the CSTR? |
|
In the CPS CSTR we have inlet and Bulk and |
|
then for now, treat it the same. |
ce48358 to
7f15645
Compare
|
So i kind of reworked state, residual equations and Initialization but it currently fails. Aiming to write the residuals equations later here. IDA is currently not converging. |
You need help with that? |
|
7f15645 to
d9dc824
Compare
d9dc824 to
650259e
Compare
|
So now a short summary about the residual equations: Retentate (rej is based on the rejection model): mw are molecular weights,
Permeate:
Cake Equations:
Permeate tank equations: Pressue equation: so the pressure equations is:
This is still work in Progress. I highly doubt, |
a63040e to
7b3eab8
Compare
85fea05 to
5b93fc4
Compare
5b93fc4 to
f73e0ef
Compare
|
Yeah so to sum it up, it seems to converge but wether the solution does make sense or not i can't say yet. |
|
Could you please move the figure and the corresponding equations to your first post? That would make it easier to find the relevant information, should we ever need to come back to this PR. |
Done |

This pull request adds
c_cakeandn_cakefor theDeadEndFiltrationmodel. It also reworks the entire state structure of the unit operation as the following graphics and equations show.So now a short summary about the residual equations:$\tilde{c}_{in}$ is the connection value between uo.):
Inlet (note that
Retentate (rej is based on the rejection model):
mw are molecular weights,$\rho$ is the density
if \dot{V}_R > 1e-16:else:Permeate:
if \dot{V}_P > 1e-16:else:Cake Equations:
if V^C> 1e-16:else:Permeate tank equations:
Pressue equation:
CV is the volume of a specific component
so the pressure equations is:
if not np.sum(\dot{n}^P) < 1e-16:else: