GNU 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
Submitter: | Lee Kelvin <leeskelvin> | ||
Submitted: | Wed 08 Feb 2017 08:21:45 PM UTC | ||
Should Start On: | Wed 08 Feb 2017 12:00:00 AM UTC | Should be Finished on: | Wed 08 Feb 2017 12:00:00 AM UTC |
Category: | Warp | Priority: | 5 - Normal |
Item Group: | New feature | Status: | None |
Privacy: | Public | Percent Complete: | 0% |
Assigned to: | None | Open/Closed: | Open |
Effort: | 0.00 |
( Jump to the original submission )
Thu 19 Mar 2020 12:38:52 AM UTC, comment #6: |
Mohammad Akhlaghi <makhlaghi>![]() |
Wed 18 Mar 2020 11:56:47 PM UTC, comment #5: I was thinking of a solution to this task and inspecting the warp code and I have few questions.
|
baha |
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>![]() |
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 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>![]() |
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 <leeskelvin>![]() |
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
In principle this task goes like this:
With these steps, the final output image box becomes exactly the size that is requested by this option.
About the question of image format: If you look in `bin/warp/ui.c', you'll notice that it uses the gal_array_read_one_ch_to_type function to load the input into memory. This high-level function can even load single color channels of JPEG images as well as FITS images! So the details of the file-format are abstracted into Gnuastro's data container (gal_data_t), so you don't have to worry about it :-).