-
-
Notifications
You must be signed in to change notification settings - Fork 918
Modelling brightness of variable stars #4208
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Great PR! Please pay attention to the following items before merging: Files matching
This is an automatically generated QA checklist based on modified files. |
Great start!
For some irregular stars we can obtain the light curves from various sources (hello NASA ADS, arXiv, and printing books) and show this historical data - it have reasons for didactic and historical purposes. Of course, we can simulate some irregular process (sine wave + some random shift for each time moment) and get a syntetic light curve (it will not related to reality) - it can be acceptable for didactic purposes. And the last point - just a show info and not touch a brightness of these stars (like at the moment).
Do you planned to use data from Exoplanets plugin? |
src/core/modules/Star.hpp
Outdated
|
||
inline void getVarStarOffset(double epoch, double &offset) const // get brightness offset for variable stars (not actual brightness) | ||
{ | ||
StarId star_id; // star ID |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about StarId star_id = (getGaia() == 0) ? getHip() : getGaia();
?
Great next step! Yes, we need definitely a few types of light curves. Occultation with 1 or 2 dips, About periods and variations: I am afraid this needs some extras files to adjust. You could use some interval, so from 1800...1870, use If performance should suffer with all these extras and too many var stars now, probably this feature can be sensibly limited to 0.1mag light changes. I am less sure about brightness changes of exoplanet transits. We cannot really display millimag changes. Maybe here the exoplanet dialog should be enhanced to show the light curve for a selected object in a QChart, and just mark the current phase with a line. |
@gzotti for many periodic variables we have already catalog with data for simple light curves ;) |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
I have started initial works on modelling brightness of variable stars.
Currently the brightness are simply sinusoidal regardless the type of variables, with brightness min, max, period and epoch at max brightness from the current GVCS data file.
To-do list:
Screenshots (if appropriate):
Untitled.mov
Type of change
How Has This Been Tested?
Test Configuration:
Checklist: