Product bundles - #40
Open
mikegottlieb84 wants to merge 15 commits into
Open
Conversation
added 12 commits
August 20, 2026 21:49
…ependency-resolution run. Reusing a WormHole instance will no longer prevent a previously unavailable product from being retried later. - CODE product tests now reflect the HTTPS/S3 migration and CODE/ root paths instead of obsolete FTP/year-specific paths.
frigusgulo
reviewed
Sep 4, 2026
|
|
||
| logger = logging.getLogger(__name__) | ||
|
|
||
| _AIUB_DOWNLOAD_ROOT = "https://www.aiub.unibe.ch/download" |
Collaborator
There was a problem hiding this comment.
These should live in yaml files and go through the propery analysis center registry
frigusgulo
reviewed
Sep 4, 2026
| full_path = pool.full_path(directory) | ||
|
|
||
| def _ls(conn: "fsspec.AbstractFileSystem") -> list[str]: | ||
| if hostname.rstrip("/") == _AIUB_DOWNLOAD_ROOT: |
Collaborator
There was a problem hiding this comment.
same comment from one above.
frigusgulo
reviewed
Sep 4, 2026
| - {name: TTT, value: RAP} | ||
| - {name: PPP, value: OPS} | ||
| directory: {pattern: "CODE/{YYYY}/"} | ||
| directory: {pattern: "CODE/"} |
Collaborator
There was a problem hiding this comment.
lets double check this one
frigusgulo
reviewed
Sep 4, 2026
| - {name: TTT, value: RAP} | ||
| - {name: PPP, value: OPS} | ||
| directory: {pattern: "CODE/{YYYY}/"} | ||
| directory: {pattern: "CODE/"} |
Collaborator
There was a problem hiding this comment.
lets double check this one
frigusgulo
reviewed
Sep 4, 2026
| - {name: SMP, value: 30S} | ||
| - {name: SMP, value: 05M} | ||
| directory: {pattern: "CODE/{YYYY}/"} | ||
| directory: {pattern: "CODE/"} |
Collaborator
There was a problem hiding this comment.
lets double check this one
frigusgulo
reviewed
Sep 4, 2026
| - {name: SMP, value: 05M} | ||
| - {name: SMP, value: 15M} | ||
| directory: {pattern: "CODE/{YYYY}/"} | ||
| directory: {pattern: "CODE/"} |
Collaborator
There was a problem hiding this comment.
lets double check this one
frigusgulo
requested changes
Sep 4, 2026
frigusgulo
left a comment
Collaborator
There was a problem hiding this comment.
lets double check those CODE product directories, and and the AIUB hard coded edge-case claude artifacts.
frigusgulo
approved these changes
Sep 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR improves PRIDE PPP-AR product selection, same-day processing, failure handling, diagnostics, and output preservation.
It introduces coherent precise-product bundles so orbit, clock, ERP, bias, and optional attitude products are selected from compatible analysis-center/product families rather than independently mixing incompatible products.
Product bundle resolution
AAA)TTT)PPP)Product precedence and sources
WRDcurrent-day mixed-GNSS broadcast navigation product.WRD, followed by normal IGS/IGN/DLR daily products.Download and cache behavior
force_downloadsupport throughout dependency resolution and transport.override_products_download=Truenow refreshes mutable products that can change, such as orbit, clock, bias, ERP, navigation, and attitude products.refresh_on_force: falseand remain cached.PRIDE product compatibility diagnostics
Added advisory validation before processing:
These checks intentionally remain advisory because some constellations may have incomplete bias support while the remaining systems can still produce a useful solution.
Frequency handling
G12R12E17C27J12pdp3command construction so frequency combinations are passed as separate arguments rather than one combined string.PrideCLIConfig.Same-day processing
brdmDDD0.YYpfilename expected bypdp3.Quaternions = NONEwhenATTOBXis unavailable, preventing unnecessary fallback downloads and avoiding PRIDE’s nonportable macOSsed -ifailure.NONEas a valid satellite-attitude configuration value.PRIDE execution robustness
pdp3from launching, while other dates in a batch can continue.error:messages even whenpdp3.shexits with status zero.60.0000000seconds by carrying them into the next minute correctly."None"product filenames into generated PRIDE configurations.Preserved output artifacts
Successful PRIDE runs now retain: