askap-vast / vast-pipeline

This repository holds the code of the Radio Transient detection pipeline for the VAST project.
https://vast-survey.org/vast-pipeline/
MIT License
7 stars 3 forks source link

Add an eta-v analysis page for the source query #586

Closed ajstewart closed 2 years ago

ajstewart commented 2 years ago

@marxide I wanted to practise some python and Django stuff so I thought I'd have a go at making something for #410.

I've somewhat cobbled together some sort of solution that seems to work. How good it is I'm not sure. While it's snappy locally I'm concerned at how it would run on a full run on Nimbus though it should still be fast. I've also added png previews of measurements on the light curve hovertool - I wonder if that is faster to gather than the fits...

This is not complete but I wondered what you thought. There are a few warnings in the console but I cannot figure them out, my front-end skills weren't quite there!

Easier to show in a video, though I couldn't find a way to quickly save it at double speed to I suggest scrubbing 😅

https://user-images.githubusercontent.com/3306161/143328350-039658ec-c7c4-491f-a0ab-2600d41d20b2.mov

To do:

Fixes #410.

ajstewart commented 2 years ago

I've noticed the zooming on the bokeh plot doesn't work in the low value regions. Not sure why as of yet.

Edit: Fixed in https://github.com/askap-vast/vast-pipeline/pull/586/commits/353903e112943635ba99c91b513c1f7866c5f67b.

ajstewart commented 2 years ago

@marxide I think this is now ready. This turned into the plot and fixing the current test issues with 3.8 and 3.9 as I merged it...

To summarise, main new features and fixes are:

Final edits I'll get around to doing in another PR: