search

askap-vast / vast-tools

A collection of tools that are useful for the VAST project and for exploration of results from the VAST Pipeline.
https://vast-survey.org/vast-tools/
MIT License
8 stars 0 forks source link

Change postagestamps to use offset axis ticks/labels instead of absolute coordinates #315

Closed ddobie closed 2 years ago

ddobie commented 2 years ago

Fix #314.

pep8speaks commented 2 years ago

Hello @ddobie! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2022-02-01 02:24:57 UTC
vast-bot commented 2 years ago

Hello @ddobie! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2022-02-01 02:25:04 UTC
ddobie commented 2 years ago

I'm not completely happy with the implementation in Source.make_png but it works.

I also feel that the testing could be more robust