Opened 3 years ago

Closed 2 years ago

Last modified 2 years ago

#3483 closed task (fixed)

i.atcorr: update spectral response curves with new create_iwave.py

Reported by: sbl Owned by: sbl
Priority: normal Milestone: 7.4.2
Component: Imagery Version: svn-trunk
Keywords: i.atcorr Cc: grass-dev@…
CPU: All Platform: All

Description

The script for creating response curves in i.atcorr (create_iwave.py) has recently been updated. All sensors for which csv files are available should be updated with the improved script (see also: #3482)

Relevant sensors are:

  • AVNIR.csv
  • geoeye1.csv
  • ikonos.csv
  • landsat_8.csv
  • pleiades1a.csv
  • pleiades1b.csv
  • PRISM-B.csv
  • PRISM-F.csv
  • PRISM-N.csv
  • quickbird2.csv
  • rapideye.csv
  • sentinel_2A_msi.csv
  • spot5.2
  • spot5.3
  • spot5.4
  • spot5.5
  • spot6.csv
  • spot7.csv
  • VGT1_spot4.csv
  • VGT2_spot5.csv
  • worldview2.csv
  • worldview3.csv

Change History (12)

comment:1 Changed 3 years ago by neteler

Cc: grass-dev@… added

Please keep grass-dev@… in cc when editing tickets!

Last edited 3 years ago by neteler (previous) (diff)

comment:2 in reply to:  description Changed 3 years ago by mmetz

Replying to sbl:

The script for creating response curves in i.atcorr (create_iwave.py) has recently been updated. All sensors for which csv files are available should be updated with the improved script (see also: #3482)

Relevant sensors are:

  • AVNIR.csv
  • geoeye1.csv
  • ikonos.csv
  • landsat_8.csv
  • pleiades1a.csv
  • pleiades1b.csv
  • PRISM-B.csv
  • PRISM-F.csv
  • PRISM-N.csv
  • quickbird2.csv
  • rapideye.csv
  • sentinel_2A_msi.csv
  • spot5.2
  • spot5.3
  • spot5.4
  • spot5.5
  • spot6.csv
  • spot7.csv
  • VGT1_spot4.csv
  • VGT2_spot5.csv
  • worldview2.csv
  • worldview3.csv

Finished with trunk r72148. Changes to WorldView? 2 where backported to relbr74 because the order of bands was wrong in iwave.cpp. Changes to WorldView? 3 where backported to relbr74 because of excessive zeroes in the spectral responses. All in relbr74 r72149.

comment:3 Changed 3 years ago by martinl

Milestone: 7.5.07.6.0

Milestone renamed

comment:4 Changed 3 years ago by mlennert

Do I understand correctly that r72148 only deals with WorldView? images ? What about the other sensors ?

comment:5 Changed 3 years ago by sbl

Yes, you are right. The other sensors are on my toDo list...

comment:6 in reply to:  5 Changed 3 years ago by mmetz

Replying to sbl:

Yes, you are right. The other sensors are on my toDo list...

Note that all sensors have been updated in trunk and only WorldView2 has been updated in relbr74 because the order of bands was mixed up. The spectral responses in iwave.cpp for the other sensors could be backported to relbr74.

comment:7 Changed 3 years ago by sbl

Milestone: 7.6.07.4.1

Ah, OK, great! (I should have checked the history and not only the commits you referenced here...)
Re-targeting the ticket, since only back-porting seems missing...

comment:8 Changed 2 years ago by neteler

Milestone: 7.4.17.4.2

comment:9 Changed 2 years ago by sbl

Has this been backported and can this ticket be closed?

comment:10 Changed 2 years ago by neteler

Resolution: fixed
Status: assignedclosed

I have compared trunk and relbr74, they are in line (except for the new Planet support in trunk: r73024, r73027).

Closing.

comment:11 in reply to:  10 ; Changed 2 years ago by mmetz

Replying to neteler:

I have compared trunk and relbr74, they are in line (except for the new Planet support in trunk: r73024, r73027).

PlanetScope? support has already been added to relbr74 with r73025,6. Maybe your local copy of relbr74 is not up to date?

comment:12 in reply to:  11 Changed 2 years ago by neteler

Replying to mmetz:

PlanetScope? support has already been added to relbr74 with r73025,6. Maybe your local copy of relbr74 is not up to date?

Oh, indeed (/me under-coffee'ed?). Thanks for the clarification.

Note: See TracTickets for help on using tickets.