Michael Kelley

Results 99 comments of Michael Kelley

Yeah, they should get removed. I think the only drawback is that older cached files would fail. When was the change made? It might also require an update of the...

If not fully parse it, I would like to at least see this text in the metadata of the table returned to the user.

Unfortunately, removing the blackslashes causes Sphinx to interpret those lines as parts of an enumerated list, so "(2001) Einstein" becomes "2001. Einstein". However, we can fix that by following the...

To be done: rebase after #383 is merged. Update change log.

Those reported lines don't make sense. They show up as covered in the codecov app: https://app.codecov.io/gh/NASA-Planetary-Science/sbpy/pull/392/blob/sbpy/spectroscopy/core.py

I've rebased. Maybe that will fix them? If not, I think the coverage report should take precedence.

I can refresh the codecov check and it'll report 100% covered as it should. But the diff report always lists those lines as uncovered. Anyway, something to keep in mind...