I need this script to loop through a folder of images on my desktop on windows and change the pixels and format from .sid to .tif using Gdal.It wont run can use please help. I have cygwin on my pc but this is my first time using this program.

Hello there and welcome.

I'm afraid I don't know about cygwin, but I just tried your script and it runs fine. Just a suggestion, you don't need ls to list files in the for loop, bash does filename expansion:

Oh, another thing. If your filenames contain spaces, quote the variables:

I'm afraid python doesn't understand bash syntax. Use bash instead. Perhaps this thread will help you.

1 members found this post helpful.

Thanks. So now I can stop wasting my time in Python. I guess the real problem is that I am new to programming. Do you have any suggestions on how I can run the script instead of using Cygwin.

When i ran the script in Cygwin it gives me the following:

>#!/bin/bash
"#!" is not recognized as an internal or external command operable program or batch file.

>x=0
"x" is not recognized as an internal or external command operable program or batch file. f was unexpected at this time.

>for f in *.sid; do

Did you read this thread? Maybe that's all you need, give it a try.

Thanks Diantre I did. I received the errors above.

Why use ls, why not just "*.sid"? Using ls just makes things more complicated and breaks support for filenames with spaces.

Also, avoid backticks, usr $(command) instead. It's easy to nest and can't be confused with single quotes.

Finally, there's no need to use expr. bash can do arithmetic:

Did you intentionally comment out the commands? Why did you write text in front of them?

What makes you think you can run a bash script with Python? That makes about as much sence as trying to compile a Python program with a C compiler. Run it like this:

Thanks MTK358

I found out that I didn't have the complete Cygwin package downloaded so I re-ran the script as is above.


Now it is saying
line 4: $'\r': command not found
line 9: gdal_translate:command not found
Can someone please help me correct this problem?

Again, you don't need ls. It just makes the script more complex and makes it fail when there are spaces in filenames. Just replace it with this:

You're not running the command here, you're just setting the varaible x to the string "expr $x+1". Also, you need to separate the "$x", "+", and "1" parts into separate arguments. Use this instead:

Or even better, use bash's built-in arithmetic. It's better because it doesn't need to launch another process:

I made the changes and I am still getting the same outputs.

Try running this command and then running the script again:

1 members found this post helpful.

I see you marked the above post as helpful. Did it fix your problem?

Anyway, what is does is convert Windows newlines (CRLF) to Unix newlines (LF) by removing all the CR characters from the file. When you write scripts, it's probably a good idea to either use a Unix editor or a Windows editor that lets you set it to use Unix newlines.

Yes it did. Thanks again. Now I am trying to figure out how to get Cygwin to read the gdal command gdal_translate.
Also today I learned to use dos2unix and I downloaded Vim to convert.

Locate where (in which folder) you have put gdal, gdal_translate etc.

Change the script to prefix the folder name to the gdla_command:
/folder_name_of_gdal/gdal_translate -of GTiff -outsize 31238 39915 "$f" "res_$f"

OK

1 members found this post helpful.