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

first implementation of easy def model #9

Merged

Conversation

daklauss
Copy link
Member

@daklauss daklauss commented Jul 29, 2024

First implementation of the easy def model as described in #5, stil wip

TODOS:

  • Tests
    • Residual tests
    • Test unit Operation
  • Considering outsourcing to rejection base class?

schmoelder and others added 5 commits June 24, 2024 11:06
Added residual and testfiles

Removed false risidual, + cstr residual v<0 => error

Seperated error testing in another class

Throw not implemented error @Viscosity

Notinitialized Error now inherits from CADETPython
SimulatorError
happening

adding a pipeline.yml for testing with pytest on push

adding ruff.yml wip

addded a ruff workflow

added python 3.12 to the pipeline to check when
sundials and numpy are working
residuals + unit operation still need testing
@daklauss daklauss requested a review from schmoelder July 29, 2024 14:15
@daklauss daklauss self-assigned this Jul 29, 2024
@daklauss daklauss linked an issue Jul 29, 2024 that may be closed by this pull request
Not all tests where found, now all are executed .... but not all finish
@schmoelder schmoelder force-pushed the master branch 2 times, most recently from 112dadf to e83f534 Compare July 30, 2024 14:50
daklauss and others added 10 commits July 31, 2024 09:32
cadet process and fixes the small bug with the
molecular weight, added molecular volume

first implementation of the def unit_operation
and residual equations still

both still need testing and unit_operation def
doesn't have rejection model yet
CAKEVOLUME and PRESSURE DROP

Adjusted tests for unit_operation to the new
CPSComponentsystem
Added residual and testfiles

Removed false risidual, + cstr residual v<0 => error

Seperated error testing in another class

Throw not implemented error @Viscosity

Notinitialized Error now inherits from CADETPython
SimulatorError
happening

adding a pipeline.yml for testing with pytest on push

adding ruff.yml wip

addded a ruff workflow

added python 3.12 to the pipeline to check when
sundials and numpy are working
residuals + unit operation still need testing
Not all tests where found, now all are executed .... but not all finish
cadet process and fixes the small bug with the
molecular weight, added molecular volume

first implementation of the def unit_operation
and residual equations still

both still need testing and unit_operation def
doesn't have rejection model yet
CAKEVOLUME and PRESSURE DROP

Adjusted tests for unit_operation to the new
CPSComponentsystem
@schmoelder
Copy link
Contributor

Hey @daklauss could you please rebase onto master?

@daklauss daklauss merged commit 19fd1e7 into master Aug 1, 2024
0 of 3 checks passed
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

Successfully merging this pull request may close these issues.

Add model equations for simple Dead-End Filtration Model
2 participants