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

Add dead end filtration #8

Merged
merged 2 commits into from
Jul 18, 2024
Merged

Add dead end filtration #8

merged 2 commits into from
Jul 18, 2024

Conversation

daklauss
Copy link
Member

Closes #7

@daklauss daklauss requested a review from schmoelder July 18, 2024 11:09
@daklauss daklauss self-assigned this Jul 18, 2024
@daklauss daklauss linked an issue Jul 18, 2024 that may be closed by this pull request
4 tasks
@schmoelder
Copy link
Contributor

Before merging, please clean up the commit history a bit. If you want, I can also do it.

@daklauss
Copy link
Member Author

Before merging, please clean up the commit history a bit. If you want, I can also do it.

done

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
@daklauss daklauss merged commit aeb139b into master Jul 18, 2024
0 of 6 checks passed
@daklauss daklauss deleted the add-dead-end-filtration branch July 18, 2024 12:33
daklauss added a commit that referenced this pull request Aug 1, 2024
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.

Restructure residual calculation + Cstr - [opened]
2 participants