Skip to content

Conversation

@Mesh-ach
Copy link
Collaborator

@Mesh-ach Mesh-ach commented Dec 17, 2025

Adjusted the model card filepath

from

/Volumes/{env_schema}/{databricksify_inst_name(query_result[0][0].name)}_gold/gold_volume/model_cards/{run_id}/model-card-{model_name}.pdf

to

/Volumes/{env_schema}/{databricksify_inst_name(query_result[0][0].name)}_gold/gold_volume/model_cards/model-card-{model_name}.pdf


@Mesh-ach Mesh-ach marked this pull request as ready for review December 17, 2025 18:40
@Mesh-ach
Copy link
Collaborator Author

@vishpillai123 Unfortunately we can't merge this into staging, without merging all the EDA changes too

@Mesh-ach
Copy link
Collaborator Author

@vishpillai123 Unfortunately we can't merge this into staging, without merging all the EDA changes too

Or do we have a way of testing this on dev? Do we have model cards for schools on dev?

@vishpillai123
Copy link
Contributor

@vishpillai123 Unfortunately we can't merge this into staging, without merging all the EDA changes too

Or do we have a way of testing this on dev? Do we have model cards for schools on dev?

Yes you can test this now! Try USCB. You could actually try any school. I've backfilled all of them.

And about the merging, yeah you're right we are blocked. Let's bring this up during scrum.

@Mesh-ach
Copy link
Collaborator Author

@vishpillai123 Unfortunately we can't merge this into staging, without merging all the EDA changes too

Or do we have a way of testing this on dev? Do we have model cards for schools on dev?

Yes you can test this now! Try USCB. You could actually try any school. I've backfilled all of them.

And about the merging, yeah you're right we are blocked. Let's bring this up during scrum.

I'll get this into develop and test, thank you.

@Mesh-ach Mesh-ach merged commit ff5dc68 into develop Dec 17, 2025
5 checks passed
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