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.

Sign up for GitHub

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

Jump to bottom

RAMSES key error #1847

Closed
cphyc opened this issue Jun 21, 2018 · 2 comments · Fixed by #1881
Closed

RAMSES key error #1847

cphyc opened this issue Jun 21, 2018 · 2 comments · Fixed by #1881
Labels
bug code frontends Things related to specific frontends help wanted Contributions from external projects' maintainers is particularly appreciated !
Milestone
3.5

Comments

@cphyc
Copy link
Member

cphyc commented Jun 21, 2018

Bug report

Bug summary

When creating a derived field for a RAMSES dataset that depends on hydro and rt (any combination as long as the fields aren't stored in the same file), a KeyError is raised. This is due to the fact that the code is trying to read all the fields from the same file.

Code for reproduction

This requires ramses_rt_00088. See https://gist.github.com/cphyc/dc445db37e09bf2a89adf50409866f26.

Version Information

  • Operating System: Arch linux
  • Python Version: 3.6.5
  • yt version: master
  • Other Libraries (if applicable):
@ngoldbaum
Copy link
Member

Hey Corentin, just checking do you want help with fixing this?

@cphyc cphyc added bug help wanted Contributions from external projects' maintainers is particularly appreciated ! code frontends Things related to specific frontends labels Jun 21, 2018
@cphyc
Copy link
Member Author

cphyc commented Jun 21, 2018

Hey,
I won't have time to look at this anytime soon, so if any help is welcome!

@ngoldbaum ngoldbaum added this to the 3.5 milestone Jun 21, 2018
cphyc added a commit to cphyc/yt that referenced this issue Jul 8, 2018
@cphyc
Dont loop twice over generator object
4f65846 
This fixes yt-project#1847
@cphyc cphyc mentioned this issue Jul 8, 2018
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug code frontends Things related to specific frontends help wanted Contributions from external projects' maintainers is particularly appreciated !
Projects
None yet
Milestone
3.5
Development

Successfully merging a pull request may close this issue.

[Bugfix] Dont loop twice over generator object cphyc/yt
2 participants
@ngoldbaum @cphyc