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

V2 dev diff daniele #46

Closed
wants to merge 53 commits into from
Closed

Conversation

dhamm97
Copy link
Collaborator

@dhamm97 dhamm97 commented Mar 8, 2023

Module for PDE-based diffusion. This is a pre-PR, tests are missing and some refactoring is foreseen.

The following classes represent the building blocks that can be used to assemble a diffusion operator:

  • _BalloonForce()
  • _Diffusivity()
  • _DiffusionCoeffcient()

Daughter classes of the above implement specific balloon forces, diffusivities and diffusion coefficients (more will be added).

These building blocks can be combined to obtain elements of the class _DiffusionOp.

The user is meant to use one of the following daughter classes of _DiffusionOp:

  • DivergenceDiffusionOp()
  • SnakeDiffusionOp()
  • TraceDiffusionOp()
  • CurvaturePreservingDiffusionOp()

Currently, in the documentation we exhibit all classes and methods, private or not. This is because important bits of the documentation can be found in currently private classes. Final refactoring (of the doc too) will deal with this differently.

joanrue and others added 26 commits April 24, 2023 13:52
- made FiniteDifference and GaussianDerivative private
- consistenT notations throughout
- clarification of finite difference and gaussian derivative operators definition
@dhamm97 dhamm97 force-pushed the v2-dev-diff_daniele branch from b872549 to 912e3e8 Compare April 28, 2023 09:51
@joanrue joanrue deleted the branch pyxu-org:v2-dev-diff May 23, 2023 07:36
@joanrue joanrue closed this May 23, 2023
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.

3 participants