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.

Sign up for GitHub

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

Jump to bottom

various bug fixes #94

Merged
merged 8 commits into from Jun 4, 2023
Merged

various bug fixes #94

merged 8 commits into from Jun 4, 2023

Conversation

johntruckenbrodt
Copy link
Member

  • get_config: strip formatting characters like \r from date strings (mindate, maxdate), which were present when directly passing the arguments via a command line call in bash scripts (12c8e33)
  • group_by_time: compute the absolute acquisition time difference to decide whether two scenes go into the same group (a7a149a)
  • processor.main: do not reassign variable inside loop (c3d5bf9)
  • processor.main: do not raise an error when no scenes were found but just print a message and quit instead (8d30451)
  • added the sensor to the log file configuration overview (b9e78ef)
  • STACArchive: retry opening and searching the catalog to address temporary unavailability or increased latency (a2e511e)
  • STACArchive.select: filter out duplicates (4d033d9)

@johntruckenbrodt
[config.get_config] strip formatting characters from date string
12c8e33 
\r was present after passing mindate via command line in a bash script
@johntruckenbrodt
[ancillary.group_by_time] compute absolute time difference
a7a149a 
scenes with a negative time difference were put into the same group
@johntruckenbrodt
[processor] do not reassign variable inside loop
c3d5bf9 
did not result in an error but nonetheless good to fix
@johntruckenbrodt
[processor] when no scenes were found, print a message and quit inste…
8d30451 
…ad of raising an error
@johntruckenbrodt
[ancillary._log_process_config] also log the sensor in the processor …
b9e78ef 
…configuration
@johntruckenbrodt
[STACArchive] retry opening and searching the catalog
a2e511e 
The catalog was found to occasionally not be reachable and the pystac-client library does not yet offer an option to increase timeout or retry.
However, this is about to change in later versions: stac-utils/pystac-client#520.
@johntruckenbrodt
[STACArchive] filter out duplicates
4d033d9 
Some scenes are reprocessed if something went wrong during initial processing.
The scenes have a different unique identifier in the name, e.g.:
S1A_IW_GRDH_1SDV_20220120T052632_20220120T052657_041540_04F0B2_1E29.SAFE
S1A_IW_GRDH_1SDV_20220120T052632_20220120T052657_041540_04F0B2_D7C6.SAFE

Now, a mechanism detects duplicates and only selects the scene with the latest processing time.
@johntruckenbrodt
added filelock to doc building dependencies
9ed67b3
@johntruckenbrodt johntruckenbrodt merged commit 9ece8be into main Jun 4, 2023
1 check passed
@johntruckenbrodt johntruckenbrodt deleted the bugfix branch June 4, 2023 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@johntruckenbrodt