Skip to content

Conversation

@anth-volk
Copy link
Contributor

Fixes #125.

What Changed

  • Added a utility function to fetch the version of a given package in an environment
  • Added a function to fetch the version of a policyengine country package based upon country ID
  • Updated EconomyComparison schema to include country_package_version key
  • Updated calculate_economy_comparison() to provide the country package version key as part of its returned EconomyComparison dict
  • Added tests for utility functions

Why This Change

  • Allows for passing of country package version value to workflow run output, simplifying downstream comparison of this and v1 workflow country package version

Copy link
Collaborator

@mikesmit mikesmit left a comment

Choose a reason for hiding this comment

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

One opinionated comment, otherwise nitts or questions.

@anth-volk
Copy link
Contributor Author

Update: tests fail due to a typo in the schema definition

@anth-volk anth-volk marked this pull request as draft May 13, 2025 17:31
@anth-volk
Copy link
Contributor Author

Modifying based on review. Switching to draft until complete.

Copy link
Collaborator

@mikesmit mikesmit left a comment

Choose a reason for hiding this comment

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

One opinionated comment, otherwise nitts or questions.

@anth-volk anth-volk marked this pull request as ready for review May 13, 2025 20:38
@anth-volk anth-volk merged commit 8d821cb into main May 13, 2025
3 checks passed
@anth-volk anth-volk deleted the feat/api-version-in-output branch May 13, 2025 21:12
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.

Add country package version to economy output runs

3 participants