Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix cases where floating point numbers are used as array indices. #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Infinoid
Copy link

Python (v3.6.9) fails to execute this code, with errors like the following:

examples$ python3 AFRL_demo.py 
Traceback (most recent call last):
  File "AFRL_demo.py", line 27, in <module>
    [phs, platform] = phsRead.AFRL(directory, pol, start_az, n_az = 3)
  File "../ritsar/phsRead.py", line 76, in AFRL
    R_c = pos[npulses/2]
IndexError: only integers, slices (`:`), ellipsis (`...`), numpy.newaxis (`None`) and integer or boolean arrays are valid indices

This fixes those errors for all of the examples.

@jks7592
Copy link

jks7592 commented Apr 2, 2020

I put the changes you listed that I didn't already have in my fork. Let me know if you see anything else, I've made quite a few changes in trying to get AFRL's Wide Angle SAR to work for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants