rice-solar-physics / pydrad

Python tools for setting up HYDRAD runs and parsing output
https://pydrad.readthedocs.io
MIT License
4 stars 3 forks source link

Follow new format for magnetic and B fields + misc. #135

Closed wtbarnes closed 3 years ago

wtbarnes commented 3 years ago

Fixes #119

Also provides a fix for outputting the loop length (and other quantities) to a sane number of significant digits. In some cases, this was causing segmentation faults in the initial conditions code (see rice-solar-physics/HYDRAD#82)

Also revised the default setup so that it can be run out of the box.

sjbradshaw commented 3 years ago

I’ll give this a go and, if it works for me, will close my issue.

Cheers!

From: Will Barnes notifications@github.com Sent: Friday, December 11, 2020 11:14 PM To: rice-solar-physics/pydrad pydrad@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: [rice-solar-physics/pydrad] Follow new format for magnetic and B fields + misc. (#135)

Fixes #119 https://github.com/rice-solar-physics/pydrad/issues/119

Also provides a fix for outputting the loop length (and other quantities) to a sane number of significant digits. In some cases, this was causing segmentation faults in the initial conditions code (see rice-solar-physics/HYDRAD#82 https://github.com/rice-solar-physics/HYDRAD/issues/82 )

Also revised the default setup so that it can be run out of the box.


You can view, comment on, or merge this pull request online at:

https://github.com/rice-solar-physics/pydrad/pull/135

Commit Summary

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/rice-solar-physics/pydrad/pull/135 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACC6C7QLJ2RC7MQ4IKNJPY3SUL3ZZANCNFSM4UXWW3VA . https://github.com/notifications/beacon/ACC6C7SFTVAAZLXE4GYKA63SUL3ZZA5CNFSM4UXWW3VKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4LL7FYQQ.gif