grd349 / PBjam

A repo for our peak baggin code and tips on jam
MIT License
17 stars 6 forks source link

Issue#199 #203

Closed grd349 closed 4 years ago

grd349 commented 4 years ago

The start to the adding the l=1 odyssey. I have added a GUI.

Still to do for just the GUI to be useful ...

nielsenmb commented 4 years ago
grd349 commented 4 years ago

@nielsenmb Good point.

I was thinking an intermediate implementation would be the sliders to do the current asy_peakbag job but for the l=1. We could dev all the peakbag l=1 work but based on human intervention. Then at some point we transition to replacing the sliders (Reggae) with a proper algo. Make any sense?

nielsenmb commented 4 years ago

Yup! That's what I was thinking.

It's a pretty nice teaching tool though, at least I got a much better sense of what goes on today when we were looking at it. So it's definitely worth shining it up a bit later.

On Mon, 2 Dec 2019 at 19:53, Guy Davies notifications@github.com wrote:

@nielsenmb https://github.com/nielsenmb Good point.

I was thinking an intermediate implementation would be the sliders to do the current asy_peakbag job but for the l=1. We could dev all the peakbag l=1 work but based on human intervention. Then at some point we transition to replacing the sliders (Reggae) with a proper algo. Make any sense?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/grd349/PBjam/pull/203?email_source=notifications&email_token=AEJWO326K3MOT6VMB6XR4DTQWVRSDA5CNFSM4JTNZ2Y2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFUWBTA#issuecomment-560554188, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEJWO366KI7UENM3AMLT72TQWVRSDANCNFSM4JTNZ2YQ .

grd349 commented 4 years ago

Ironman sucks

grd349 commented 4 years ago
Screenshot 2019-12-02 at 20 52 52
grd349 commented 4 years ago
Screenshot 2019-12-02 at 21 06 53
grd349 commented 4 years ago

I have been looking at epsilon_g for our star of choice (Boeing 4448777). Is there any theoretical justification for having an epsilon_g that isn't zero. I'm not talking about about XYZ fitted stuff to models (screw models ;) ) but rather does epsilon_g actually mean anything? What might epsilon_g vary with in terms of fundamental parameters?

grd349 commented 4 years ago

OK - we need another meeting to start making a plan. @nielsenmb should we chat next week?

nielsenmb commented 4 years ago

Sure! I'm free any day expect Friday 6th.

grd349 commented 4 years ago

Reggae is now integrated (not fully but getting there) with the star class and even has an implementation in plotting (plot_echelle). I want to improve the integration a bit and fix a few of the niggles. Then once we have docs, comments, tests, and examples in the docs we can think about merging.

If you want to try it on your favourite star (4448777 works best as some numbers are hard coded for now) then just star.run_reggae() after you have run the asy_fit. You can even call star.reggae.plot_echelle().

grd349 commented 4 years ago

I also need to add pyqt5 to the requirements.

nielsenmb commented 4 years ago

Awesome! So it opens the UI from the star class?

Also having Reggae output a frequency list in some form is useful for some upcoming work if it doesn't already do this.

grd349 commented 4 years ago

Yes - star.run_reggae() is now a thing!

It outputs some frequencies and then you can star.reggae.plot_echelle()

Screenshot 2020-02-18 at 04 14 43
grd349 commented 4 years ago

I'm going to close this PR as the cpnest PR has all of this stuff in there and is much more advanced.