-
-
Notifications
You must be signed in to change notification settings - Fork 404
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
interpolate outside of 2000 and 50000 setting zeta to 1. #3
Comments
@wkerzendorf, @ssim: I guess that this refers to the temperature grid for the zeta values...Not sure how the PR #395 relates to this....Is the interpolation actually an issue or kept for documentation purposes. If it's the latter, let's close this issue and put a note on the docu. |
PR #395 is completely unrelated to this. However, in the course of the plasma overhaul, zetas were automatically set to 1 whenever the temperatures outside the grids are encountered. See @staticmethod
def get_zeta_values(zeta_data, ion_index, t_rad):
zeta_t_rad = zeta_data.columns.values.astype(np.float64)
zeta_values = zeta_data.ix[ion_index].values.astype(np.float64)
zeta = interpolate.interp1d(zeta_t_rad, zeta_values, bounds_error=False,
fill_value=np.nan)(t_rad)
zeta = zeta.astype(float)
if np.any(np.isnan(zeta)):
warnings.warn('t_rads outside of zeta factor interpolation'
' zeta_min={0:.2f} zeta_max={1:.2f} '
'- replacing with 1s'.format(
zeta_data.columns.values.min(), zeta_data.columns.values.max(),
t_rad))
zeta[np.isnan(zeta)] = 1.0
return zeta Thus, this issue is resolved - closing! |
Merge pull request #3 from KevinCawley/master
Merge pull request #3 from smithis7/master
No description provided.
The text was updated successfully, but these errors were encountered: