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

Problem with ion populations in LegacyPlasmaArray #380

Closed
ssim opened this issue Aug 17, 2015 · 4 comments
Closed

Problem with ion populations in LegacyPlasmaArray #380

ssim opened this issue Aug 17, 2015 · 4 comments

Comments

@ssim
Copy link
Contributor

ssim commented Aug 17, 2015

Hello,

I'm playing around a bit with the GUI and am repeatedly running into the following error:

Traceback (most recent call last):
  File "/Users/stuartsim/Tardis/tardis/build/lib.macosx-10.10-x86_64-2.7/tardis/gui/widgets.py", line 753, in on_atom_header_double_clicked
    self.table2_data = self.parent.model.plasma_array.ion_populations[
  File "/Users/stuartsim/Tardis/tardis/build/lib.macosx-10.10-x86_64-2.7/tardis/plasma/base.py", line 28, in __getattr__
    super(BasePlasma, self).__getattribute__(item)
AttributeError: 'LegacyPlasmaArray' object has no attribute 'ion_populations'

I'm not 100% clear what the LegacyPlasmaArray is doing, but I assume this is something to do with the recent changes. Can you help, @aoifeboyle ?

In my configuration file I'm currently using:

plasma:
    ionization: nebular
    excitation: dilute-lte
    radiative_rates_type: dilute-blackbody
    line_interaction_type: scatter

is that all still fine, or do I need to do something else with the new plasma?

@aoifeboyle
Copy link
Contributor

Yeah I know what it is actually! That property has been renamed ion_number_density in the new plasma. Sorry, I should have gone through the gui to correct those terms. I'll do it now.

@ssim
Copy link
Contributor Author

ssim commented Aug 17, 2015

Great - thanks!

@aoifeboyle
Copy link
Contributor

@ssim It seems to work for me now! Let me know if it works for you.

@ssim
Copy link
Contributor Author

ssim commented Aug 18, 2015

@aoifeboyle Looks good - seems to be working for me now too! Thanks for fixing. I will close this issue now.

Thanks!

@ssim ssim closed this as completed Aug 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants