Skip to content

Conversation

@msuarezcrocus
Copy link
Contributor

@jalvord1 The ACTWorkKeys bundle is ready for your review.


{% endif %}

{% if "${SCHOOL_YEAR}" != "2022" %}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is the file different only in 2022? Or is it 2022 and beyond? Like if someone sent '2023' as the school_year, should the logic below apply?

@@ -0,0 +1,148 @@
# ACTWorkKeys Assessment Bundle Documentation
Copy link
Collaborator

Choose a reason for hiding this comment

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

When we merge this, we will remove this file and the hierarchy file from the bundle itself

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If so, should I remove this link from the README file?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes please!

cert: CompositeScore
- operation: map_values
column: CompositeScore
map_file: ./seeds/certificateLevelMapping.csv
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are we sending both the composite score as-is and the mapped values? or only the mapped values? Typically we avoid mapping values because users will want to see those underlying values that are accustomed to seeing in the file itself

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm sending the mapped values. Do you mean to send these values in studenAssessment.jsonl as they are in the sample file, right?

@jalvord1
Copy link
Collaborator

@msuarezcrocus This is ready for dev/qc

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