Skip to content

Navigation Menu

Appearance settings

View all features
- BY COMPANY SIZE
  Enterprises
  Small and medium teams
  Startups
  Nonprofits
- BY USE CASE
  App Modernization
  DevSecOps
  DevOps
  CI/CD
  View all use cases
- BY INDUSTRY
  Healthcare
  Financial services
  Manufacturing
  Government
  View all industries
View all solutions
- EXPLORE BY TOPIC
  AI
  Software Development
  DevOps
  Security
  View all topics
- EXPLORE BY TYPE
  Customer stories
  Events & webinars
  Ebooks & reports
  Business insights
  GitHub Skills
- SUPPORT & SERVICES
  Documentation
  Customer support
  Community forum
  Trust center
  Partners
View all resources
- COMMUNITY
  GitHub SponsorsFund open source developers
- PROGRAMS
  Security Lab
  Maintainer Community
  Accelerator
  GitHub Stars
  Archive Program
- REPOSITORIES
  Topics
  Trending
  Collections
- ENTERPRISE SOLUTIONS
  Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  GitHub Advanced SecurityEnterprise-grade security features
  Copilot for BusinessEnterprise-grade AI features
  Premium SupportEnterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

boutproject / BOUT-dev Public

Notifications You must be signed in to change notification settings
Fork 106
Star 227

Code
Issues 225
Pull requests 42
Actions
Projects
Wiki
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Actions
Projects
Wiki
Security
Insights

Utility for setting up coordinates for a tokamak system#3046

Open

tomc271 wants to merge 99 commits intorefactor-coordinatesboutproject/BOUT-dev:refactor-coordinatesfrom

dry-up-examplesboutproject/BOUT-dev:dry-up-examplesCopy head branch name to clipboard

Conversation Commits99 (99)Checks Files changed

Open

Utility for setting up coordinates for a tokamak system#3046
tomc271 wants to merge 99 commits intorefactor-coordinatesfrom
dry-up-examples

Commits

Commits on Jan 28, 2025

Extract function tokamak_coordinates(coord, Rxy, Bpxy, hthe, I, B0, Btxy)
tomchapman
committed
Use function in more examples
tomchapman
committed
Move and rename "examples/common.hxx" to "include/bout/tokamak_coordinates.hxx"
tomchapman
committed
Use function `tokamak_coordinates` in tests
tomchapman
committed
Make function `tokamak_coordinates` provide the Coordinates object
tomchapman
committed
Bxy and dx may be Field2D or Field3D
tomchapman
committed
Make tokamak_coordinates a method on new class TokamakCoordinatesFactory
tomchapman
committed
Add constructor (taking mesh as parameter)
tomchapman
committed
Assign tokamak_coordinates_factory to variable
tomchapman
committed
Field mesh_m should be private
tomchapman
committed
Rename variable (Sign of Bp)
tomchapman
committed
Prefer const ref
tomchapman
committed
Make Rxy a member variable of TokamakCoordinatesFactory
tomchapman
committed
Get coordinates from TokamakCoordinatesFactory instead of mesh->getCoordinates()
tomchapman
committed
Rxy has to be of type Field2D
tomchapman
committed
Make Bpxy a member variable of TokamakCoordinatesFactory
tomchapman
committed
Make Btxy a member variable of TokamakCoordinatesFactory
tomchapman
committed
Make B a member variable of TokamakCoordinatesFactory
tomchapman
committed
Make hthe a member variable of TokamakCoordinatesFactory
tomchapman
committed
Make I a member variable of TokamakCoordinatesFactory
tomchapman
committed
Load values for metric quantities (from mesh) inside TokamakCoordinatesFactory constructor.
tomchapman
committed
More fixes
tomchapman
committed
Getters for Rxy, Bpxy, Btxy, Bxy, and hthe cannot be const
tomchapman
committed
Fixes to dalf3 example
tomchapman
committed
Fixes to elm_pb example
tomchapman
committed
Fix typo (double semicolons)
tomchapman
committed
Assign value to new (const) variable
tomchapman
committed
Getters for Rxy, Bpxy, Btxy, Bxy, and hthe can be const
tomchapman
committed
Fixes to elm_pb_outerloop example
tomchapman
committed
Extract method TokamakCoordinatesFactory.normalise()
tomchapman
committed
Fix TokamakCoordinatesFactory method calls in elm_pb
tomchapman
committed
Add dx to TokamakCoordinatesFactory.normalise() method
tomchapman
committed
Use extracted methods in more examples and tests
tomchapman
committed
Extract method setShearFactor()
tomchapman
committed
Restore ShearFactor setter on TokamakCoordinatesFactory
tomchapman
committed
Use extracted methods in alfven-wave example
tomchapman
committed
Extract method TokamakCoordinatesFactory.get_sign_of_bp()
tomchapman
committed
Use extracted method TokamakCoordinatesFactory.normalise() in more places
tomchapman
committed
Reorder (instantiate TokamakCoordinatesFactory before first use)
tomchapman
committed
Use new getters and setters elm_pb_outerloop
tomchapman
committed
More use of new getters and setters elm_pb_outerloop
tomchapman
committed
Get Bxy from the Coordinates object
tomchapman
committed
Bxy has to be of type Field2D, so get from TokamakCoordinatesFactory object
tomchapman
committed
Bxy is now normalised in TokamakCoordinatesFactory.normalise()
tomchapman
committed
Move b0xcv to TokamakCoordinatesFactory
tomchapman
committed
Normalise b0xcv (magnetic curvature term)
tomchapman
committed
dx is now normalised in TokamakCoordinatesFactory.normalise()
tomchapman
committed
Make make_tokamak_coordinates method take optional boolean argument shifted_metric_method
tomchapman
committed
dx is now normalised in TokamakCoordinatesFactory.normalise()
tomchapman
committed
Normalise b0xcv (magnetic curvature term) moved to normalise() method on TokamakCoordinatesFactory
tomchapman
committed
Pass arguments noshear include_curvature to make_tokamak_coordinates method
tomchapman
committed
Further fixes to `Pass arguments noshear include_curvature to make_tokamak_coordinates method`
tomchapman
committed
Remove unused method set_ShearFactor()
tomchapman
committed
More fixes
tomchapman
committed
Finish: Move b0xcv to TokamakCoordinatesFactory
tomchapman
committed
Further fixes to `Pass arguments noshear include_curvature to make_tokamak_coordinates method`
tomchapman
committed
dx is now normalised in TokamakCoordinatesFactory.normalise()
tomchapman
committed
Declare variable noshear
tomchapman
committed
Further small fixes
tomchapman
committed
Declare variable noshear
tomchapman
committed
noshear and include_curvature variables don't exist in gem example
tomchapman
committed
TokamakCoordinatesFactory instance can't be const
tomchapman
committed
include_curvature variable doesn't exist in test-drift-instability/2fluid example. Declare variable noshear
tomchapman
committed
Fix: Move b0xcv to TokamakCoordinatesFactory
tomchapman
committed
Set ShearFactor before using it
tomchapman
committed
Extract method set_shearfactor_and_curvature_term()
tomchapman
committed
clang-tidy: directly include headers
tomchapman
committed
Access b0xcv (curvature term) through getter
tomchapman
committed
Fix tests gbs and conducting-wall-mode/cwm - don't redefine coords object
tomchapman
committed
Initialise noshear to false
tomchapman
committed
Must call tokamak_coordinates_factory.normalise() before tokamak_coordinates_factory.make_tokamak_coordinates()
tomchapman
committed
Pass (optional) ShearFactor parameter to TokamakCoordinatesFactory.normalise()
tomchapman
committed
Fix mix up between the two shear factor variables
tomchapman
committed
Make variable const
tomchapman
committed
Fix: "Must call tokamak_coordinates_factory.normalise() before tokamak_coordinates_factory.make_tokamak_coordinates()"
tomchapman
committed
Fix TokamakCoordinatesFactory: Set dx
tomchapman
committed
Fix syntax typo (compound assignments operators in normalise() method)
tomchapman
committed
Remove b0xcv (curvature term) from TokamakCoordinatesFactory
tomchapman
committed
Make TokamakCoordinatesFactory.normalise() private and call in make_tokamak_coordinates() method
tomchapman
committed
Rename TokamakCoordinatesFactory getters
tomchapman
committed
Rename class and method
tomchapman
committed
Move implementation of TokamakCoordinates class out of header file
tomchapman
committed
Apply clang-format changes

tomc271
authored and
tomchapman
committed
Copy values to local variables
tomchapman
committed
Fix: Add new parameters Lbar and Bbar to make_coordinates method call
tomchapman
committed
const qualifier in parameter declaration has no effect
tomchapman
committed
Fix "Copy values to local variables"
tomchapman
committed
Make TokamakCoordinates a struct rather than a class
tomchapman
committed
Rename make_coordinates() to set_tokamak_coordinates_on_mesh() and don't return anything
tomchapman
committed
Restore mesh->getCoordinates() calls
tomchapman
committed

Commits on Feb 21, 2025

Rename struct TokamakCoordinates to TokamakOptions
tomchapman
committed
Fix naming clash
tomchapman
committed
Pass no shear condition to set_tokamak_coordinates_on_mesh() as an (optional) integer parameter rather than a boolean
tomchapman
committed
If no value is specified for dpsi, set TokamakOptions.dx to mesh.getCoordinates()->dx()
tomchapman
committed
Merge branch 'refactor-coordinates' into dry-up-examples
tomchapman
committed

Commits on Mar 18, 2025

Apply black changes

tomc271
authored and
tomchapman
committed
Move implementations to new tokamak_coordinates.cxx file
tomchapman
committed
Move TokamakOptions to `bout` namespace
tomchapman
committed
Also move implementation of normalise() method to cxx file
tomchapman
committed

Footer

© 2026 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.