Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

Tutorial example for cold plasma dielectric tensor elements #16

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

jhillairet
Copy link

@jhillairet jhillairet commented Jun 7, 2019

  • Added a first draft of how to deduce cold plasma dielectric parameters using PlasmaPy
  • Corrected 2 URLs in the .md files

@jhillairet jhillairet changed the title Corrected 2 URLs in the .md files Tutorial example for cold plasma dielectric tensor elements Jun 7, 2019
@StanczakDominik StanczakDominik self-assigned this Jun 8, 2019
Copy link
Member

@StanczakDominik StanczakDominik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @jhillairet, thanks a ton for doing this! To be honest I wasn't even aware of this sub-repository until yesterday. Sorry for the delayed response.

We'll have to think what the best place to keep these would be - we've mostly been using sphinx-gallery to render notebooks out of plain python files in the source repo with parsable comments (easier to version-control those). But obviously that's a specialized format that's pretty annoying to work with and I'm sure people would prefer to work with straight ipynb when contributing. I'm not yet sure what the best way of dealing with this is - I'll look around and figure something out.

Still, that's the technical side and on me to deal with. With regards to the tutorial, it's pretty awesome, I just have a few suggestions on how to make it even better :) Once again, thanks for doing this!

"cell_type": "markdown",
"metadata": {},
"source": [
"Before plotting them against the frequency, let's filter positive and negative values, in order to display propagative and evanscent cases "
Copy link
Member

@StanczakDominik StanczakDominik Jun 8, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo (already fixed):

Suggested change
"Before plotting them against the frequency, let's filter positive and negative values, in order to display propagative and evanscent cases "
"Before plotting them against the frequency, let's filter positive and negative values, in order to display propagative and evanescent cases "

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, as a relative layman, I would love to see a sentence describing what those cases actually are - am I right in thinking that something like this would fit?

These are solutions that respectively propagate through the plasma or are quickly attenuated (exponential damping).

as in e^(i omega t) versus e^(omega t).

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, that's just an update of the example I've made in the gallery few time ago.

Concerning the sentence, in fact that's not so direct and I should modify the text. To know if the wave is evanescent or propagative, one should calculate the wave number using the dispersion relation. That's the next step to improve the example.

physics/Magnetized Cold Plasma Properties.ipynb Outdated Show resolved Hide resolved
"cell_type": "markdown",
"metadata": {},
"source": [
"## Cold Plasma tensor elements in the rotating basis"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this spot could also use a really brief description. Am I right saying that those would be left-handed and right-handed circularly polarized waves?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes indeed.

@StanczakDominik
Copy link
Member

I'll push a few of those technical changes in a minute...

@jhillairet
Copy link
Author

A question (I didn't follow all the discussions on the mailing list or github) : is there a better way to deal with units than importing astropy.units ?

@StanczakDominik
Copy link
Member

Nope, I don't think there is one - you either make the units explicit or get warnings on assuming default ones (at least I think that's how we currently want to have it work).

@StanczakDominik StanczakDominik removed their assignment Jan 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants