taskGNU Astronomy Utilities - Tasks: task #14364, Add --size as an option to...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

task #14364: Add --size as an option to ImageWarp as an alternative to --scale

Submitted by:  Lee Kelvin <leeskelvin>
Submitted on:  Wed 08 Feb 2017 08:21:45 PM UTC  
 
Should Start On: Wed 08 Feb 2017 12:00:00 AM UTCShould be Finished on: Wed 08 Feb 2017 12:00:00 AM UTC
Category: WarpPriority: 5 - Normal
Item Group: New featureStatus: None
Privacy: PublicPercent Complete: 0%
Assigned to: NoneOpen/Closed: Open
Effort: 0.00

Thu 09 Feb 2017 02:22:23 PM UTC, comment #4:

Hi Mohammad,

Mostly I do not address comments to a particular person. The only exception to that is if I particularly want to address one of their points directly!

I will seek to change my comment writing style to suit... Forces of habit die hard. :)

Lee Kelvin <leeskelvin>
Thu 09 Feb 2017 12:47:51 PM UTC, comment #3:

That sounds reasonable, then we will stick to `--size' and `--resolution' then. Thanks for the comment.

Just as a side note, I think it would be better if we don't use letter-style greetings (at the start and end of a comment) in the posts on Savannah. This is a public discussion and we don't want it to be only between two people. When you address the comment to a particular person, other interested people on the list will be discouraged from joining reading the post (when it goes in their emails especially).

For the end of the comment, writing our name is also redundant because the author's name is kept on the left side of the post on Savannah (and as the author in the mailing list archives). So when all the people don't add their names, it just makes going up or down through the comments a little harder (too much white space after some posts and not after the others). Requiring everyone to add their name would be too much and would just make the Savannah comments longer, so I think the better option is that people don't write their names in the end of each post.

Thanks again for all the great suggestions and discussions...

Mohammad Akhlaghi <makhlaghi>
Project Administrator
Thu 09 Feb 2017 12:33:11 PM UTC, comment #2:

Hi Mohammad,

Just a small point, if you change the name from '--size' to '--tosize' it becomes inconsistent with other arguments which do not currently have a preposition. For example, scaling an image is not currently '--byscale'.

To maintain consistency, either all arguments should contain a preposition or none should.

Best,
Lee

Lee Kelvin <leeskelvin>
Thu 09 Feb 2017 02:27:18 AM UTC, comment #1:

This is a great suggestion, it is very useful.

Maybe just a change of name to `--tosize' would be less ambiguous.

Building on this great suggestion, I also suggest having a `--toresolution' option, so the user can specify the output image WCS resolution without having to check the input's resolution.

If anyone is interested in implementing this, please post a comment here and I would be happy to help ;-)...

Mohammad Akhlaghi <makhlaghi>
Project Administrator
Wed 08 Feb 2017 08:21:45 PM UTC, original submission:

In order to scale an image using ImageWarp one must currently provide a scaling value using the --scale argument. If a user wishes to scale an image to a particular output size, this necessitates that they also know the input size such that they can calculate the scaling value.

I propose the addition of a --size argument to ImageWarp which will allow the user to specify an exact output pixel size without needing to calculate the scaling value beforehand. This will speed up image processing for many users, and provides another quick means by which users may scale their imaging data.

Best regards,
Lee Kelvin

Lee Kelvin <leeskelvin>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by makhlaghi (Posted a comment)
  • -unavailable- added by leeskelvin (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    No Changes Have Been Made to This Item

    Back to the top


    Powered by Savane 3.1-cleanup1