Skip to content

Commit 57c11b7

Browse files
authored
Merge pull request #71 from opera-adt/dev
v1.0.11
2 parents 5746631 + 1f6a612 commit 57c11b7

6 files changed

Lines changed: 16 additions & 6 deletions

File tree

CHANGELOG.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,15 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
66
and this project adheres to [PEP 440](https://www.python.org/dev/peps/pep-0440/)
77
and uses [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
88

9+
## [1.0.11] - 2026-01-27
10+
11+
### Changed
12+
* Defaults for max_pre_img per burst set to defaults (4, 3, 3) in enumeration of many products and single product. This aligns with standard inputs for DIST-S1.
13+
14+
### Fixed
15+
* Casting time strings into timestamps using pandas 2.0+ was attaching precision that was breaking schema validation with different precision (I believe pandas default is nano-second precision and ASF distributes with microsecond precision). Using `coerce=True` for the schema (forcing it to cast into the correct precision) and forcing data to have UTC time-zones fixes this issue with validation.
16+
17+
918
## [1.0.10] - 2026-01-07
1019

1120
### Added

README.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,8 @@ Yields:
7676
<details>
7777
<summary>Output</summary>
7878

79-
```[{'mgrs_tile_id': '19HBD',
79+
```
80+
[{'mgrs_tile_id': '19HBD',
8081
'post_acq_date': '2023-11-05',
8182
'track_number': 91,
8283
'post_acq_timestamp': '2023-11-05 23:36:49+00:00'},

src/dist_s1_enumerator/asf.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ def get_rtc_s1_ts_metadata_by_burst_ids(
135135
properties_f = [
136136
{
137137
'opera_id': p['sceneName'],
138-
'acq_dt': pd.to_datetime(p['startTime']),
138+
'acq_dt': pd.Timestamp(p['startTime'], tz='UTC'),
139139
'track_number': p['pathNumber'],
140140
'polarizations': p['polarization'],
141141
'all_urls': [p['url']] + p['additionalUrls'],

src/dist_s1_enumerator/dist_enum.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ def enumerate_dist_s1_products(
231231
df_rtc_ts: gpd.GeoDataFrame,
232232
mgrs_tile_ids: list[str],
233233
lookback_strategy: str = 'multi_window',
234-
max_pre_imgs_per_burst: int = (5, 5, 5),
234+
max_pre_imgs_per_burst: int = (4, 3, 3),
235235
min_pre_imgs_per_burst: int = 1,
236236
tqdm_enabled: bool = True,
237237
delta_lookback_days: int = 365,

src/dist_s1_enumerator/dist_enum_inputs.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ def enumerate_dist_s1_workflow_inputs(
3232
start_acq_dt: datetime | pd.Timestamp | str | None = None,
3333
stop_acq_dt: datetime | pd.Timestamp | str | None = None,
3434
lookback_strategy: str = 'multi_window',
35-
max_pre_imgs_per_burst: int | list[int] | tuple[int, ...] = (5, 5, 5),
35+
max_pre_imgs_per_burst: int | list[int] | tuple[int, ...] = (4, 3, 3),
3636
min_pre_imgs_per_burst: int = 1,
3737
delta_lookback_days: int | list[int] | tuple[int, ...] = 365,
3838
delta_window_days: int = 365,
@@ -60,7 +60,7 @@ def enumerate_dist_s1_workflow_inputs(
6060
Lookback strategy to use, by default 'multi_window'. Options are
6161
'immediate_lookback' or 'multi_window'.
6262
max_pre_imgs_per_burst : int | list[int] | tuple[int, ...], optional
63-
Maximum number of pre-images per burst to include, by default 10.
63+
Maximum number of pre-images per burst to include, by default (4, 3, 3).
6464
If lookback strategy is 'multi_window':
6565
- this is interpreted as the maximum number of pre-images on each anniversary date.
6666
- tuple/list of integers are provided, each int represents the maximum number of pre-images on each

src/dist_s1_enumerator/tabular_models.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
{
2828
'opera_id': Column(str, required=True),
2929
'jpl_burst_id': Column(str, required=True),
30-
'acq_dt': Column(DateTime(tz='UTC'), required=True),
30+
'acq_dt': Column(DateTime(tz='UTC'), coerce=True, required=True),
3131
'acq_date_for_mgrs_pass': Column(str, required=False),
3232
'polarizations': Column(str, required=True),
3333
'track_number': Column(int, required=True),

0 commit comments

Comments
 (0)