gnuastro-devel
[Top][All Lists]
Advanced

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

[task #16545] Add region support in script-fits-view


From: Mohammad Akhlaghi
Subject: [task #16545] Add region support in script-fits-view
Date: Mon, 3 Jun 2024 14:26:54 -0400 (EDT)

Update of task #16545 (group gnuastro):

                  Status:                    None => Done                   
        Percent Complete:                      0% => 90%                    
             Open/Closed:                    Open => Closed                 

    _______________________________________________________

Follow-up Comment #2:

Thanks Giacomo, the commit has been merged as the modified
[https://git.savannah.gnu.org/cgit/gnuastro.git/commit/?id=8eb178da8 Commit
8eb178da8]. 

Here are the modifications:
* There are two ds9 commands generated in this script ('execom'), but you had
only added the new '$ds9region' string to the first one, not the second. It is
always good to check the larger context of the operation and make sure we
don't add future bugs (region files only loaded in the first scenario, not the
second).
* There was no updates to the documentation for the newly added option! Any
change in the behavior of a program should be documented in
'doc/gnuastro.texi' (as you see in the modified commit that was merged).
* Don't forget to set the "Percent complete" option in Savannah when making a
post that involves an improvement compared to the previous comment on the
bug/task.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/task/?16545>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/




reply via email to

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