Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 13:41
    namurphy labeled #1175
  • 13:41
    namurphy labeled #1174
  • 13:36
    namurphy opened #1175
  • 13:28
    namurphy edited #1174
  • 13:18
    namurphy opened #1174
  • Jun 23 00:51
    namurphy labeled #1172
  • Jun 23 00:51
    namurphy labeled #1172
  • Jun 23 00:51
    namurphy labeled #1172
  • Jun 23 00:51
    namurphy opened #1172
  • Jun 21 18:02

    pheuer on main

    Input ion velocity in thomson n… (compare)

  • Jun 16 21:07
    namurphy edited #1170
  • Jun 16 21:07
    namurphy edited #1169
  • Jun 16 21:05
    namurphy labeled #1170
  • Jun 16 21:05
    namurphy labeled #1170
  • Jun 16 21:05
    namurphy opened #1170
  • Jun 16 21:05
    namurphy labeled #1170
  • Jun 16 20:58
    namurphy labeled #1169
  • Jun 16 20:58
    namurphy labeled #1169
  • Jun 16 20:58
    namurphy labeled #1169
  • Jun 16 20:58
    namurphy opened #1169
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
I'd be in favor of immutability for CustomParticle as well; if it's ever needed needed for time-dependent stuff, we can make a custom object for that
Nick Murphy
@namurphy:matrix.org
[m]
Making CustomParticle immutable does sound reasonable to me. Right now it's mutable.
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Sounds just right for the dataclass refactor, then!
I'm asking because I've had a few ideas for IonizationState's (and IonicLevel's) immutability after falling down a rabbit hole on hashability. Basically, if you want something to be hashable (thus, be a dict key), you should keep it immutable. I'm sidestepping the issue by using the ionic symbol as a key for now.
Nick Murphy
@namurphy:matrix.org
[m]
To me it seems reasonable to have IonizationState be mutable since that's something that often ends up changing over time.
Come to think of it...we might start thinking of immutability and mutability in terms of whether it makes sense physically.
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Yeah, that sounds reasonable enough - unless you represented a series of time snapshots by a bunch of IonizationState objects.
Nick Murphy
@namurphy:matrix.org
[m]
Yeah, that would be another possibility. Hm...
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Well, it's not something I'm suggesting we change overnight :D
Nick Murphy
@namurphy:matrix.org
[m]
True!
Oh wait I mean True!
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
yes
Nick Murphy
@namurphy:matrix.org
[m]
Changing CustomParticle to be immutable would be a good first step. I'll create an issue on that right now.
Particle is hashable, it's set up to have the same hash as the particle symbol too.
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Makes sense!
Nick Murphy
@namurphy:matrix.org
[m]
Yep I totally did it because it makes sense and not because I was having too much fun with Python
Would making CustomParticle immutable have advantages when using, say, Numba?
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Probably not ☹️
At least not AFAIK
I was thinking about Peter's issue with particles and scipy optimize
I have no idea whether immutability would help out with that, though
Nick Murphy
@namurphy:matrix.org
[m]
It'd be worth giving some more thought to whether or not there are other use cases for keeping them mutable. I can't think of many at the moment...which could either be because there aren't many cases, or because of pandemic brain.
And also — thank you for bringing this up! This has also been in the back of my mind lately too.
Dominik Stańczak
@StanczakDominik:matrix.org
[m]

Another wacky IonizationState question! We can currently do something like this:

all_species = IonizationStateCollection(
    {
        "H": [0, 1],
        "C": [0, 0, 0, 0, 0, 0, 1],
    },
    n0=1e20 * u.m ** -3,
    abundances={"H": 1, "C": 0.08},
    T_e=10 * u.eV,
)

And that represents, well, a blob. But... suppose we could put arrays in? Even 1D arrays would be pretty helpful for my usage. I'm going to need 1D profiles (for each flux surface) of each ion's density and temperature for thesis purposes. I could either stick the 1D arrays in IonizationState or I could build each IonizationState from 1D arrays of parameters in a for loop. Of course, the issue then is that I'm looping at the pure python level.

Dominik Stańczak
@StanczakDominik:matrix.org
[m]
We can now use https://www.plasmapy.org/chat to link back to this room. Idea lifted with love from the sunnier side of python. ;)
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Probably a good idea to also add https://www.plasmapy.org/jitsi while I'm at it! Does anyone have any more redirect suggestions?
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Here's a reminder that we've got our weekly community meeting in 40min from now, on https://plasmapy.org/jitsi and with the minutes over at https://hackmd.io/@plasmapy/BJSrFenUd :)
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
:point_up: Edit: Here's a reminder that we've got our weekly community meeting in 40min from now, on https://www.plasmapy.org/jitsi and with the minutes over at https://hackmd.io/@plasmapy/BJSrFenUd :)
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
#1122, the worst kind of bugfix, where things work and I have no clue why.
Github
@_neb_github:matrix.org
[m]
PlasmaPy/PlasmaPy#1122 : Pickleable particles
Nick Murphy
@namurphy:matrix.org
[m]
We're having problems with Jitsi so we're going to the office hours zoom backup room for today's meeting now
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
I just found out about https://cherab.github.io/documentation/welcome.html and this looks very, very, very relevant.
StanczakDominik
@StanczakDominik:matrix.org
[m]
Ping Julien Hillairet , the above looks particularly interesting for you given your recent question about accessing atomic data
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Also ping Lemmatum about the above, for spectroscopic reasons
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Friendly reminder, community meeting, right now, plasmapy.org/jitsi
Nick Murphy
@namurphy:matrix.org
[m]
Running slightly late...
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
no worries!
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
RTD builds seem to have been failing for a while, ever since sphinx 4.0 got released with some deprecations. I'll pin a lower version for now and try to find a proper fix on the weekend.
1 reply
Also the hypothesis tests seem a little flaky but I have a workaround in mind. Will handle both today.
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
I'll have to skip tomorrow's hack week meeting for first 💉 dose reasons - but on the flip side, this means it's review day today 😀
Nick Murphy
@namurphy:matrix.org
[m]
That is a completely justified excuse! Hopefully the side effects aren't too bad!
1 reply
And a friendly reminder that we are having PlasmaPy office hours starting in 15 minutes at 2 pm EDT / 18 UTC. Please feel free to stop by!
Dominik Stańczak
@StanczakDominik:matrix.org
[m]

Hey everyone, if you run into 'nbformat.notebooknode.NotebookNode object' has no attribute 'tags' on local doc builds, spatialaudio/nbsphinx#563; update nbsphinx to 0.85 (which reqs jinja2 < 3) to prevent that.

Made me actually learn virtualenvs. 😤 😅

G. Zhou
@g_zhou:matrix.org
[m]
I tried to reproduce the "Particle stepper" in the Example on my personal computer. However, the drift speed I got by "ParticleTracker" is opposite to the drift speed calculated by "ExB_drift". The output result is "The calculated drift velocity is 0.5150 m / s to compare with the expected E0/B0 = -0.5000 V / (m T)".
I am sure that there is no problem with the code because I copied the code in the document directly.
3 replies
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Friendly reminder that we've got our weekly community meeting at plasmapy.org/jitsi in 15 minutes, just enough time to make some tea beforehand! with the minutes at https://hackmd.io/@plasmapy/rJFIp_ZFO/edit :)
Dominik Stańczak
@StanczakDominik:matrix.org
[m]
Friendly reminder that we're due for a meeting at plasmapy.org/jitsi in 45 minutes (enough time to make plenty of tea beforehand); with the minutes at https://hackmd.io/@plasmapy/Bk5jv1mFd !
Nick Murphy
@namurphy:matrix.org
[m]
I just changed the default branch name for PlasmaPy repositories on GitHub to main, following some discussions at our community meetings. When you go to a repository now, some instructions will pop up for what needs to be done for local repositories. GitHub has automatically updated pull requests, etc. to reflect the branch name change.
Nick Murphy
@namurphy:matrix.org
[m]
We're having our weekly community meeting in about ten minutes at plasmapy.org/jitsi with the minutes and agenda at https://hackmd.io/6AAncRKmRHij97Owx4_3xA
(Also I had some issues with not being able to edit the agenda hackmd doc for some reason, but it seemed to start working if I kept the tab open for a minute or so. Not sure what was happening with that.)