[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[gnuastro-commits] master 8be4edd7 18/39: Zeropoint: a bug about the mag
From: |
Mohammad Akhlaghi |
Subject: |
[gnuastro-commits] master 8be4edd7 18/39: Zeropoint: a bug about the magnitude range |
Date: |
Wed, 19 Apr 2023 12:18:24 -0400 (EDT) |
branch: master
commit 8be4edd7211022e80b6c5871cc954661932ddc5e
Author: Elham Saremi <saremi_elham@yahoo.com>
Commit: Mohammad Akhlaghi <mohammad@akhlaghi.org>
Zeropoint: a bug about the magnitude range
Until now, if the user used the option --magnituderange with two values
minimum and maximum of magnitude, there was no problem. But if the user
didn't determine this range, an error is encountered. While this option is
not mandatory.
With this commit, the bug is fixed. The bug was related to setting minimum
and maximum magnitude on the header of output while there was no condition
for when users don't use the range.
---
bin/script/zeropoint.mk | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/bin/script/zeropoint.mk b/bin/script/zeropoint.mk
index 6fd16433..c8dc76d5 100644
--- a/bin/script/zeropoint.mk
+++ b/bin/script/zeropoint.mk
@@ -270,8 +270,12 @@ $(zeropoint): $(aperzeropoint)
astfits $@.fits --write=ZPAPER,"$$bestaper","Best aperture."
astfits $@.fits --write=ZPVALUE,"$$bestzp","Best zeropoint."
astfits $@.fits --write=ZPSTD,"$$beststd","Best standard deviation of
zeropoint."
- astfits $@.fits --write=MAGMIN,"$$magmin","Minimum magnitude for
obtaining zeropoint."
- astfits $@.fits --write=MAGMAX,"$$magmax","Maximum magnitude for
obtaining zeropoint."
+
+# If the user requested a certain magnitude range, add minmag and maxmag
to header.
+ if [ x"$(magrange)" != x ]; then
+ astfits $@.fits --write=MAGMIN,"$$magmin","Minimum magnitude for
obtaining zeropoint."
+ astfits $@.fits --write=MAGMAX,"$$magmax","Maximum magnitude for
obtaining zeropoint."
+ fi
if [ x"$(keepzpap)" = x ]; then
- [gnuastro-commits] master updated (dd7ccfb7 -> 21cd71bf), Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master 1ddcf29a 06/39: zeropoint: options of zeropoint estimation is written in book, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master c3fc5214 04/39: Zeropoint: add options with variable, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master d44ce8c0 10/39: zeropoint: use --magrage provided by user, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master 8be4edd7 18/39: Zeropoint: a bug about the magnitude range,
Mohammad Akhlaghi <=
- [gnuastro-commits] master 10030cfc 11/39: Zeropoint: add magnitude range into the header of output, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master e39330ab 17/39: Zeropoint: adding sanity check for aperture option, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master 46d328cd 12/39: Book: writing a tutorial for the zero-point script; preface, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master 01dcbeed 15/39: Book: tutorial of zero point; magnitude range and aperture size, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master b3077f0d 27/39: Book: revision of the first part of the zeropoint tutorial, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master 17b5fa81 36/39: Book: simplifying and polishing the zero point script tutorial, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master 7769e279 07/39: zeropoint: new options for magnitude and keeping the results are added, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master a1c7a277 16/39: Book: tutorial of zero point; select the best aperture, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master 1a987e04 19/39: Book: tutorial of zero point; using keepzpap and completion of results, Mohammad Akhlaghi, 2023/04/19
- [gnuastro-commits] master a3c0f0e2 31/39: Zeropoint: optimize the script for overlaping check, Mohammad Akhlaghi, 2023/04/19