Skip to content

Comments

Refresh main.py#43

Open
dylanlee wants to merge 2 commits intomainfrom
update-entrypoint
Open

Refresh main.py#43
dylanlee wants to merge 2 commits intomainfrom
update-entrypoint

Conversation

@dylanlee
Copy link
Contributor

@dylanlee dylanlee commented Aug 28, 2025

The changes in this PR are the changes made to the main.py entrypoint in src during PI7 autoeval data production. Main changes are:

  1. Add aoi_is_item flag. This allows us to specify use the aoi tag value as a stac_item_id and bypass spatial querying of the benchmark_stac.
  2. Add more exhaustive pipeline failure reporting and make it so that more things make a pipeline explicitely fail. This was necessary to save computation time on pipelines that were failing for valid reasons.
  3. Early exit logic when no flow scenarios or catchments are found. This logs when a pipeline exits early and doesn't produce data but isn't a failure.

Add aoi_is_item flag

Add more exhaustive pipeline failure reporting and make it so that more
things make a pipeline explicitely fail.

Early exit logic when no flow scenarios or catchments are found
Copy link
Member

@ar-siddiqui ar-siddiqui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR has changes forced by the formatter as well. Please change back your formatter to the original setting and reformat files.

@dylanlee
Copy link
Contributor Author

dylanlee commented Sep 9, 2025

Okay this is because I had to create a new workstation and hadn't backed up the original formatting settings. Do you remember where you listed the formatting rules you want to follow?

@dylanlee
Copy link
Contributor Author

The line lengths have been updated

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