-
Notifications
You must be signed in to change notification settings - Fork 3
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
325 observation details are always strings #356
Conversation
…ther than string; example of integer property now in DASCH (flags)
Fixed typo (Phometry -> Photometry) in comment
Hi @dbenn , |
…SS*, ZTF plug-ins.
Hi @dbenn , |
Looks good @mpyat2. Do you have comments on my code on this branch? |
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.
Hi @dbenn
sorry for delay. Looks good.
…FLUX, DET_FLUX_ERR instead of KSPSAP_FLUX_ERR. The plug-in was updated to be compatible with the new format.
Hi @dbenn , |
@mpyat2 I think we're OK to merge this now. Agreed? I have a working Mac again BTW. :) |
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.
Thanks @mpyat2
Hi @dbenn , |
Hi @mpyat2
There are quite a few changes on this branch and I'll have to sort out merge conflicts. The branch has changes from #317 and #325.
EDIT: The conflict was in
plugin/src/org/aavso/tools/vstar/external/plugin/ASASObservationSource.java
so could you please check that I have not messed anything up there? Seems OK.David