Skip to content

Performance provision quality dlp#32

Draft
kena-SL wants to merge 7 commits intomainfrom
performance-provision-quality-DLP
Draft

Performance provision quality dlp#32
kena-SL wants to merge 7 commits intomainfrom
performance-provision-quality-DLP

Conversation

@kena-SL
Copy link
Copy Markdown

@kena-SL kena-SL commented May 15, 2025

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

This PR creates a shell script to automate the generation and upload of provision-quality dataset.
The build-performance script executes the generate_provision_quality Python function and uploads the resulting Parquet file to S3.
Uploads output Parquet to s3://${COLLECTION_DATASET_BUCKET_NAME}/performance/provision-quality/.
Validates presence of the output file and required environment variable before uploading.

Note: requirements.txt points to a branch in digital-land-python for testing purpose

Related Tickets & Documents

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above. Please refer to the Digital Land Testing Guidance for more information.

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

[optional] Are there any post deployment tasks we need to perform?

[optional] Are there any dependencies on other PRs or Work?

Yes

Comment thread task/requirements.txt
@@ -1 +1 @@
-e git+https://github.com/digital-land/pipeline.git@main#egg=digital-land
-e git+https://github.com/digital-land/pipeline.git@provision-quality-dataset#egg=digital-land No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we need to change this

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.

3 participants