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 BRL Langmuir probe theory #445

Open
lemmatum opened this issue May 7, 2018 · 3 comments · May be fixed by #2344
Open

Add BRL Langmuir probe theory #445

lemmatum opened this issue May 7, 2018 · 3 comments · May be fixed by #2344
Labels
plasmapy.diagnostics Related to the plasmapy.diagnostics subpackage
Milestone

Comments

@lemmatum
Copy link
Contributor

lemmatum commented May 7, 2018

BRL (Bernstein-Rabinowitz-Laframboise) is often used in comparison with OML (orbital motion limited) and ABR (Allen-Boyd-Reynolds) theories for Langmuir probes.

@lemmatum lemmatum added the plasmapy.diagnostics Related to the plasmapy.diagnostics subpackage label May 7, 2018
@StanczakDominik StanczakDominik modified the milestones: v0.1.1, v0.2.0 May 8, 2018
@StanczakDominik StanczakDominik modified the milestones: v0.2.0, Future Oct 16, 2018
@kuchtact
Copy link

I'd love to start working on implementing the BRL theory into PlasmaPy as I don't see any implementations of it elsewhere. Unfortunately the full BRL theory from Laframboise's thesis is quite bulky so I'm not sure of the direct use of the raw theory. I think it should still be implemented but it might not be the most useful for quickly fitting curves. Chen has an algebraic curve that approximates BRL for probe potential less than plasma potential which is here: F. F. Chen, “Langmuir probe analysis for high density plasmas,” Physics of Plasmas, vol. 8, no. 6, pp. 3029–3041, Jun. 2001, doi: 10.1063/1.1368874. That could be more useful but isn't the true theory.

I'd like some input as to whether implementing the full BRL theory would be useful. If so, I can get started writing some code!

@namurphy
Copy link
Member

@rocco8773 — do you have any thoughts on this issue and @kuchtact's comment?

@rocco8773
Copy link
Member

I say have at it. For now, put it into plasmapy.diagnostics. Depending on how expansive the code gets you (@kuchtact) can either create an appropriate file (e.g. brl.py) or a sub-directory (e.g. brl/). Once the functionality gets more hashed out, we can refactor the package/file structure appropriately.

@kuchtact kuchtact linked a pull request Sep 21, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plasmapy.diagnostics Related to the plasmapy.diagnostics subpackage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants