Skip to content

Max bille 251127 update simulation comments (#3)#303

Closed
MaxBille wants to merge 1 commit intolettucecfd:masterfrom
MaxBille:master
Closed

Max bille 251127 update simulation comments (#3)#303
MaxBille wants to merge 1 commit intolettucecfd:masterfrom
MaxBille:master

Conversation

@MaxBille
Copy link
Copy Markdown
Collaborator

  • Update _simulation.py with comments

clarification of what simulation is doing for new and experienced users dealing with boundaries.

  • Update _simulation.py for understandability (add1)

  • Update _simulation.py - comment lines wrap v1.1

  • re-wrapped and re-wrote comments to keep them within the 72-79 character window, recommended by PEP8

Description

Checklist

  • This pull request is associated to an issue
  • This PR contains a description
  • Did you add a new method? If so, you need to
    • add method description
    • maybe mention class in the corresponding __init__
    • add an example using the method in examples/advanced_flows/ or examples/simple_flows/
    • add a test in tests/
  • Add someone else as reviewer and wait for approval before merging.

* Update _simulation.py with comments 

clarification of what simulation is doing for new and experienced users dealing with boundaries.

* Update _simulation.py for understandability (add1)

* Update _simulation.py - comment lines wrap v1.1

* re-wrapped and re-wrote comments to keep them within the 72-79 character window, recommended by PEP8
@MaxBille MaxBille requested a review from PhiSpel March 16, 2026 14:18
@PhiSpel
Copy link
Copy Markdown
Contributor

PhiSpel commented Mar 16, 2026

duplicate of #289

@PhiSpel PhiSpel closed this Mar 16, 2026
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.

2 participants