gnuastro-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[gnuastro-commits] master 87913fc5: Fits: SPACE printed between outputs


From: Mohammad Akhlaghi
Subject: [gnuastro-commits] master 87913fc5: Fits: SPACE printed between outputs of --skycoverage
Date: Thu, 28 Dec 2023 07:02:08 -0500 (EST)

branch: master
commit 87913fc5693675855c0f5854ef54da5321cb8cbc
Author: Mohammad Akhlaghi <mohammad@akhlaghi.org>
Commit: Mohammad Akhlaghi <mohammad@akhlaghi.org>

    Fits: SPACE printed between outputs of --skycoverage
    
    Until now, when the Fits program was run with the '--skycoverage' option
    without '--quiet', if the precision of one dimention was high (as in data
    cubes with angstroms in the third dimension), it would connect the numbers
    for the minimum and maximum value along that dimension.
    
    With this commit, the issue has been solved by adding a SPACE character
    between the formatted print that generated them.
    
    This bug was found with the help of Rahna Payyasseri Thanduparackal.
    
    This fixes bug #65084.
---
 NEWS                         |  2 ++
 THANKS                       |  1 +
 bin/fits/fits.c              | 10 +++++-----
 doc/announce-acknowledge.txt |  1 +
 4 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/NEWS b/NEWS
index 65d9995c..b338e39b 100644
--- a/NEWS
+++ b/NEWS
@@ -159,6 +159,8 @@ See the end of the file for license conditions.
   - bug #65050: Use --libtool option in the TEMPLATE usage; found and fixed
     by Faezeh Bidjarchian.
   - bug #65055: Columns of empty FITS tables (with no rows) read in reverse.
+  - bug #65084: Fits program's --skycoverage prints connected numbers for
+    cubes; found with help of Rahna Payyasseri Thanduparackal.
 
 
 
diff --git a/THANKS b/THANKS
index a9713424..82f25387 100644
--- a/THANKS
+++ b/THANKS
@@ -107,6 +107,7 @@ support in Gnuastro. The list is ordered alphabetically (by 
family name).
     Paul Eggert                          eggert@cs.ucla.edu
     Peter Teuben                         teuben@umd.edu
     Pierre-Alain Duc                     pierre-alain.duc@astro.unistra.fr
+    Rahna Payyasseri Thanduparackal      rpayyasseri@cefca.es
     Raphael Morales                      rmorales@iaa.es
     Rashid Yaaqib                        r.yaaqib@gmail.com
     Raúl Infante Sainz                   infantesainz@gmail.com
diff --git a/bin/fits/fits.c b/bin/fits/fits.c
index 44f9f363..76447aaa 100644
--- a/bin/fits/fits.c
+++ b/bin/fits/fits.c
@@ -559,13 +559,13 @@ fits_skycoverage(struct fitsparams *p)
       switch(ndim)
         {
         case 2:
-          printf("  Center: %-15.10g%-15.10g\n", center[0], center[1]);
-          printf("  Width:  %-15.10g%-15.10g\n", width[0],  width[1]);
+          printf("  Center: %-15.10g %-15.10g\n", center[0], center[1]);
+          printf("  Width:  %-15.10g %-15.10g\n", width[0],  width[1]);
           break;
         case 3:
-          printf("  Center: %-15.10g%-15.10g%-15.10g\n", center[0],
+          printf("  Center: %-15.10g %-15.10g %-15.10g\n", center[0],
                  center[1], center[2]);
-          printf("  width:  %-15.10g%-15.10g%-15.10g\n", width[0],
+          printf("  width:  %-15.10g %-15.10g %-15.10g\n", width[0],
                  width[1], width[2]);
           break;
         default:
@@ -579,7 +579,7 @@ fits_skycoverage(struct fitsparams *p)
                        0, 0, &nwcs, "--hdu");
       printf("\nSky coverage by range along dimensions:\n");
       for(i=0;i<ndim;++i)
-        printf("  %-8s %-15.10g%-15.10g\n", gal_wcs_dimension_name(wcs, i),
+        printf("  %-8s %-15.10g %-15.10g\n", gal_wcs_dimension_name(wcs, i),
                min[i], max[i]);
       wcsfree(wcs);
     }
diff --git a/doc/announce-acknowledge.txt b/doc/announce-acknowledge.txt
index 1153c23c..0cd3c9a2 100644
--- a/doc/announce-acknowledge.txt
+++ b/doc/announce-acknowledge.txt
@@ -1,6 +1,7 @@
 Alphabetically ordered list to acknowledge in the next release.
 
 Ignacio Ruiz Cejudo
+Rahna Payyasseri Thanduparackal
 Sepideh Eskandarlou
 Teet Kuutma
 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]