Fixed a bug introduced with the addition of Orion star particle support: if your orion dataset has no star particles, this just warns you about it and happily continues on.
update: isolated reading of orion particles into a method. this is the result of a ridiculous transpose made due to my inability to use hg, so it's possible some changeset revisions changed numbers. not clear if this is a bad thing or not...
update 2: fixed a bug Andrew Myers pointed out.
Hi Jeff. This looks okay, but I would request that instead of wrapping inside a try/except block, the entire star particle reader be put into a subroutine, since it doesn't seem to affect the state of the rest of the routine. Then it could return none if not os.exists(...).
You might also want to take a look at 63be95c6afcc in @alrosen's fork of yt which handles sink particles as well as star particles.
J. S. Oishiauthor
This does indeed take into account 63be95c6afcc, but I had to do a transplant, and it renumbered it. I hope this all works ok, but I fear it will not.
I think that _readOrionParticleFile needs to return "True" or 1 or something, but other than that, this looks good to me. Thanks for the fix!